Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

ci: update test script #1

Merged
merged 11 commits into from
Aug 8, 2024
Merged

ci: update test script #1

merged 11 commits into from
Aug 8, 2024

Conversation

hollandjg
Copy link
Member

@hollandjg hollandjg commented Jul 18, 2024 •
edited
Loading

Update workflow script for testing:

  • Change dependencies to python 3.11+ – don't bother about older versions.
  • Use pytest (rather than tox)
  • Add pytest dependency to test extra

@hollandjg hollandjg marked this pull request as ready for review July 18, 2024 20:00
@hollandjg hollandjg changed the title CI: update test script ci: update test script Jul 19, 2024
@hollandjg hollandjg requested review from jashlu and zzaydin August 5, 2024 17:27
zzaydin
zzaydin suggested changes Aug 5, 2024
View reviewed changes
.github/workflows/test.yml Outdated Show resolved Hide resolved
.idea/inspectionProfiles/profiles_settings.xml Outdated Show resolved Hide resolved
.idea/social-norms-trees.iml Outdated Show resolved Hide resolved
@hollandjg hollandjg changed the base branch from main to delete-idea August 5, 2024 18:56
@hollandjg hollandjg requested a review from zzaydin August 5, 2024 19:01
@hollandjg hollandjg changed the base branch from delete-idea to main August 5, 2024 19:04
@hollandjg hollandjg changed the base branch from main to delete-idea August 5, 2024 19:12
Base automatically changed from delete-idea to main August 7, 2024 15:33
zzaydin
zzaydin approved these changes Aug 7, 2024
View reviewed changes
Copy link
Collaborator

@zzaydin zzaydin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already went through similar changes, just updating versions!

@hollandjg hollandjg merged commit 9542247 into main Aug 8, 2024
6 checks passed
@hollandjg hollandjg deleted the update-test-script branch August 8, 2024 12:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zzaydin zzaydin zzaydin approved these changes

@jashlu jashlu Awaiting requested review from jashlu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hollandjg @zzaydin