Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated github actions #502

Merged
merged 5 commits into from
Jan 23, 2024
Merged

Updated github actions #502

merged 5 commits into from
Jan 23, 2024

Conversation

nishaq503
Copy link
Collaborator

This PR:

  • updates the package-filter action to:
    • succeed even when the source branch is not up-to-date with the target branch
    • find all updated packages and outputs their directories
    • test if there is always a VERSION file to accompany an updated pyproject.toml file
    • test that every updated package has a dev version
    • test that at least one package was updated
  • updates the tests action to run pre-commit hooks and pytest on all updated packages
  • updates the package-release action to:
    • bump the release version for all updated packages
    • make a single commit for bumping version in all those packages

@Nicholas-Schaub Nicholas-Schaub merged commit 4ff8a79 into PolusAI:master Jan 23, 2024
1 of 2 checks passed
hamshkhawar pushed a commit to hamshkhawar/image-tools that referenced this pull request Jan 25, 2024
@nishaq503 @hamshkhawar
Updated github actions (PolusAI#502)
888df46 
* ci: updated plugin-filter to no longer rely on 3rd party plugin and also getting all changed packages

* ci: Updated tests and release workflows

* ci: actions now correctly checkout repo

* fix: checking out full repo

* fix: tests fowkflow uses CUR_DIR correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nicholas-Schaub Nicholas-Schaub Awaiting requested review from Nicholas-Schaub Nicholas-Schaub is a code owner

@NHotaling NHotaling Awaiting requested review from NHotaling NHotaling is a code owner

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
@nishaq503 @Nicholas-Schaub