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

Add diff/log commands #150

Merged
merged 7 commits into from
Mar 11, 2024
Merged

Add diff/log commands #150

merged 7 commits into from
Mar 11, 2024

Conversation

PierreLoupG
Copy link
Contributor

Add two new commands, diff and log.

The diff command shows the current revision differences of all sources compared to the referenced revision in the menu.

$ cooker diff
poky: dunfell-23.0.0 .. dunfell-23.0.13
meta-openembedded: ab9fca4 .. 321dc39

The log command prints the changes (added, modified, deleted) of the build sources between two menu versions. The formatted output can be appended to a changelog file for release.

$ cooker log qemux86 menu.old.json -H meta-openembedded -o md
## Added projects
- meta-d at revision 895fac4

## Modified projects
- meta-openembedded changed from ab9fca4 to 321dc39
  - 321dc39b3 commit message
  - ef65938fd commit message
- meta-b changed from 5d2abf6 to 37f332a

## Deleted projects
- meta-common at revision 0c560e7

Test for the log command added in the test-suit.

Pierre-loup Gosse added 6 commits March 4, 2024 16:17
Show current revision changes of all sources compared to the referenced
source from menu file.
Generates build sources revision changes structure between two menu file
version.
@pboettch
Copy link
Collaborator

Looks good. Could you please run pylint on your changed and see whether all corrections make sense? I saw, e.g., 2 empty lines between classes and functions are not there.

@pboettch pboettch self-requested a review March 11, 2024 10:35
@cpb- cpb- merged commit 0eecc01 into cpb-:master Mar 11, 2024
1 check passed
@cpb-
Copy link
Owner

cpb- commented Mar 11, 2024

Thanks Pierre-Loup !

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.

None yet

3 participants