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

dev: delete extra file when running git-submodule --update #764

Merged

Conversation

giuseppe-steduto
Copy link
Member

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 #761.

Copy link

codecov bot commented Dec 6, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (09c9210) 18.61% compared to head (e5680ce) 18.61%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #764   +/-   ##
=======================================
  Coverage   18.61%   18.61%           
=======================================
  Files          26       26           
  Lines        2326     2326           
=======================================
  Hits          433      433           
  Misses       1893     1893           
Files Coverage Δ
reana/reana_dev/git.py 29.25% <ø> (ø)

@giuseppe-steduto giuseppe-steduto marked this pull request as ready for review December 6, 2023 13:31
Copy link
Member

@tiborsimko tiborsimko left a 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.
Copy link
Member

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."

@giuseppe-steduto giuseppe-steduto force-pushed the git-submodule-delete-when-updating branch from 8cedf29 to 8721571 Compare December 7, 2023 08:31
Copy link
Member

@mdonadoni mdonadoni left a 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

Copy link
Member

@mdonadoni mdonadoni left a 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
Comment on lines 4 to 9
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.

Copy link
Member

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

@giuseppe-steduto giuseppe-steduto force-pushed the git-submodule-delete-when-updating branch from 97b2d07 to 70e1bb1 Compare January 17, 2024 15:00
giuseppe-steduto added a commit to giuseppe-steduto/reana that referenced this pull request Jan 17, 2024
…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.
@giuseppe-steduto giuseppe-steduto force-pushed the git-submodule-delete-when-updating branch from 70e1bb1 to 267c4ae Compare January 17, 2024 15:01
…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.
@giuseppe-steduto giuseppe-steduto force-pushed the git-submodule-delete-when-updating branch from 267c4ae to e5680ce Compare January 18, 2024 08:55
@mdonadoni mdonadoni merged commit e5680ce into reanahub:master Jan 18, 2024
15 checks passed
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.

reana-dev git-submodule does not remove deleted files from destination
3 participants