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.

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

feat: Add option to delete preexisting files created by dfg each run #32

Merged
merged 4 commits into from
Feb 23, 2023

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Feb 23, 2023

This adds the ability to request that dfg erase any files it previously created before generating new ones. This logic may be used to replace cleanup logic already present in the tests as well as logic currently embedded in the GHA tool currently running dfg. Moving this logic directly into dfg will allow us to use its pre-commit hook as the primary source of truth for how to run it, both locally and in CI.

Contributes to rapidsai/shared-workflows#46

@vyasr vyasr added feature request New feature or request non-breaking Introduces a non-breaking change labels Feb 23, 2023
@vyasr vyasr self-assigned this Feb 23, 2023
@vyasr vyasr changed the title Add option to delete preexisting files created by dfg each run feat: Add option to delete preexisting files created by dfg each run Feb 23, 2023
@vyasr
Copy link
Contributor Author

vyasr commented Feb 23, 2023

Note: I have intentionally not modified the pre-commit hook to always include the --clean argument. Even if we do end up wanting that behavior for every repo, I think that's better specified in their pre-commit configs rather than hardcoding it here.

Copy link
Member

@ajschmidt8 ajschmidt8 left a comment

Choose a reason for hiding this comment

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

Approved with one minor suggestion.

Looks great.

@vyasr vyasr merged commit 381a835 into rapidsai:main Feb 23, 2023
@vyasr vyasr deleted the feat/delete_old_files branch February 23, 2023 17:54
ajschmidt8 pushed a commit that referenced this pull request Feb 23, 2023
# [1.4.0](v1.3.1...v1.4.0) (2023-02-23)

### Features

* Add option to delete preexisting files created by dfg each run ([#32](#32)) ([381a835](381a835)), closes [/github.com/rapidsai/gha-tools/blob/37f3946294647eadf5b60d8229ac396a4d913b59/tools/rapids-dependency-file-checker#L9-L15](https://github.com//github.com/rapidsai/gha-tools/blob/37f3946294647eadf5b60d8229ac396a4d913b59/tools/rapids-dependency-file-checker/issues/L9-L15)
@ajschmidt8
Copy link
Member

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

difyrrwrzd added a commit to difyrrwrzd/dependency-file-generator that referenced this pull request Aug 10, 2024
# [1.4.0](rapidsai/dependency-file-generator@v1.3.1...v1.4.0) (2023-02-23)

### Features

* Add option to delete preexisting files created by dfg each run ([#32](rapidsai/dependency-file-generator#32)) ([10e5118](rapidsai/dependency-file-generator@10e5118)), closes [/github.com/rapidsai/gha-tools/blob/37f3946294647eadf5b60d8229ac396a4d913b59/tools/rapids-dependency-file-checker#L9-L15](https://github.com//github.com/rapidsai/gha-tools/blob/37f3946294647eadf5b60d8229ac396a4d913b59/tools/rapids-dependency-file-checker/issues/L9-L15)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Introduces a non-breaking change released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants