-
Notifications
You must be signed in to change notification settings - Fork 56
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
dev: delete extra file when running git-submodule --update #764
dev: delete extra file when running git-submodule --update #764
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #764 +/- ##
=======================================
Coverage 18.61% 18.61%
=======================================
Files 26 26
Lines 2326 2326
=======================================
Hits 433 433
Misses 1893 1893
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Left cosmetic comments for the release notes.
CHANGES.rst
Outdated
-------------------------- | ||
|
||
- Developers: | ||
- Changes `git-submodule --update` command to delete from the cluster `moduels` folder the files that are not present in the shared modules directories. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Note the typo (
modules
). - Suggestion: end the phrase with "... that are not present anymore in the shared modules source directories."
8cedf29
to
8721571
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well!
Two things left to do:
- Rebase the PR
- Change the commit message to be compliant with conventional commits
8721571
to
97b2d07
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR needs to be rebased again, sorry! Plus the line in the changelog can be removed.
CHANGES.rst
Outdated
Version 0.9.3 (UNRELEASED) | ||
-------------------------- | ||
|
||
- Developers: | ||
- Changes `git-submodule --update` command to delete from the cluster `modules` folder the files that are not present anymore in the shared modules directories. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to update the CHANGELOG now that we have moved to release-please
97b2d07
to
70e1bb1
Compare
…nahub#764) Change the `git-submodule --update` command to delete from the `modules` folder of the cluster components the files that are not present in the original directories, reflecting a more accurate status of the shared modules. Closes reanahub#761.
70e1bb1
to
267c4ae
Compare
…hub#764) Change the `git-submodule --update` command to delete from the `modules` folder of the cluster components the files that are not present in the original directories, reflecting a more accurate status of the shared modules. Closes reanahub#761.
267c4ae
to
e5680ce
Compare
Change the
git-submodule --update
command to delete from themodules
folder of the cluster components the files that are not present in the
original directories, reflecting a more accurate status of the shared
modules.
Closes #761.