Skip to content

Remove run test file#72

Merged
sbillinge merged 2 commits intodiffpy:cookiefrom
bobleesj:cookie-remove-test-file
Aug 12, 2024
Merged

Remove run test file#72
sbillinge merged 2 commits intodiffpy:cookiefrom
bobleesj:cookie-remove-test-file

Conversation

@bobleesj
Copy link
Copy Markdown
Contributor

Do we need the run.py files anymore? I removed them. It works with the new main.yml file.

@bobleesj
Copy link
Copy Markdown
Contributor Author

@bobleesj bobleesj marked this pull request as ready for review August 12, 2024 23:30
@sbillinge
Copy link
Copy Markdown
Contributor

If it works,

Should I make a PR to remove tests/run.py in cookiecutter? @sbillinge

https://github.com/Billingegroup/cookiecutter/blob/main/%7B%7B%20cookiecutter.repo_name%20%7D%7D/src/%7B%7B%20cookiecutter.package_dir_name%20%7D%7D/tests/run.py

yes, please do.

@sbillinge sbillinge merged commit 06fa935 into diffpy:cookie Aug 12, 2024
@bobleesj bobleesj deleted the cookie-remove-test-file branch August 12, 2024 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants