Skip to content

Conversation

AdriiiPRodri
Copy link
Contributor

Context

This pull request removes the steps related to the installation and management of the ‎xmlsec system dependencies from the ‎api-pull-request.yml GitHub Actions workflow. The motivation for this change is to streamline the workflow by eliminating unnecessary dependencies that are no longer required for the project’s operation.

Description

The following changes have been made:

  • All steps responsible for installing ‎libxml2-dev, ‎libxmlsec1-dev, ‎libxmlsec1-openssl, and ‎pkg-config via ‎apt-get have been removed from the workflow.
  • The step that previously updated the ‎poetry.lock file after a branch name change has been consolidated and simplified.
  • The workflow now focuses on updating ‎poetry.lock only when there are relevant changes, and ensures this process is consistent for both pull request and push events.

Checklist

API

  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, Poetry, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@AdriiiPRodri AdriiiPRodri requested a review from a team as a code owner August 7, 2025 14:23
@github-actions github-actions bot added the github_actions Pull requests that update GitHub Actions code label Aug 7, 2025
@AdriiiPRodri AdriiiPRodri added the no-merge Please, DO NOT MERGE this PR. label Aug 7, 2025
Copy link
Contributor

github-actions bot commented Aug 7, 2025

✅ All necessary CHANGELOG.md files have been updated. Great job! 🎉

@AdriiiPRodri AdriiiPRodri added the no-changelog Skip including change in changelog/release notes label Aug 7, 2025
Copy link

codecov bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.22%. Comparing base (675698a) to head (9d3d5cc).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #8482       +/-   ##
===========================================
+ Coverage   75.19%   89.22%   +14.02%     
===========================================
  Files          72      198      +126     
  Lines        4725    16283    +11558     
===========================================
+ Hits         3553    14528    +10975     
- Misses       1172     1755      +583     
Flag Coverage Δ
api 94.95% <ø> (?)
prowler 75.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
prowler 75.19% <ø> (ø)
api 94.95% <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AdriiiPRodri AdriiiPRodri changed the title fix(actions): lock poetry after changes ref(actions): remove xmlsec step Aug 7, 2025
@AdriiiPRodri AdriiiPRodri requested a review from a team as a code owner August 7, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/api github_actions Pull requests that update GitHub Actions code no-changelog Skip including change in changelog/release notes no-merge Please, DO NOT MERGE this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants