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(core22): maven plugin #4657

Merged
merged 18 commits into from Mar 21, 2024
Merged

feat(core22): maven plugin #4657

merged 18 commits into from Mar 21, 2024

Conversation

cmatsuoka
Copy link
Contributor

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Copy link

codecov bot commented Mar 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.37%. Comparing base (b8c9a17) to head (b28c0da).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4657      +/-   ##
==========================================
+ Coverage   88.35%   88.37%   +0.02%     
==========================================
  Files         327      328       +1     
  Lines       21997    22046      +49     
==========================================
+ Hits        19435    19483      +48     
- Misses       2562     2563       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

Looks good. We should also add a snapcraft-specific example (the one currently in craft-parts)

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka cmatsuoka requested a review from tigarmo March 13, 2024 19:09
Copy link
Contributor

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

The build is failing because that /common/craft-parts/mave_plugin.rst shared doc doesn't exist on that version of craft-parts. We should use the dev branch just to ensure the build is successful

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
requirements-devel.txt Outdated Show resolved Hide resolved
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
This prevents duplicate label errors.

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Copy link
Contributor

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

thanks!

@sergiusens
Copy link
Collaborator

tests are broken

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@sergiusens sergiusens changed the title tests: add spread test for the maven plugin feat(core22): maven plugin Mar 21, 2024
@sergiusens sergiusens merged commit 952ecb1 into main Mar 21, 2024
11 checks passed
@sergiusens sergiusens deleted the CRAFT-2584-Document-maven-plugin branch March 21, 2024 18:37
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

5 participants