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.

Sign up for GitHub

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

Jump to bottom

docs: document release process #11210

Merged
merged 5 commits into from
Jan 9, 2025
Merged

docs: document release process #11210

merged 5 commits into from
Jan 9, 2025

Conversation

alexgarel
Copy link
Member

  • add helper script

@alexgarel
docs: document release process
20e33a4 
+ add helper script
@alexgarel alexgarel requested a review from a team as a code owner January 8, 2025 14:26
@github-actions github-actions bot added the 📚 Documentation Documentation issues improve the project for everyone. label Jan 8, 2025
stephanegigandet
stephanegigandet reviewed Jan 8, 2025
View reviewed changes
Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Very good idea to have this documentation

@codecov-commenter
Copy link

codecov-commenter commented Jan 8, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.22%. Comparing base (765d796) to head (2526f56).
Report is 12 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #11210      +/-   ##
==========================================
+ Coverage   49.18%   49.22%   +0.04%     
==========================================
  Files          78       78              
  Lines       22389    22405      +16     
  Branches     5371     5374       +3     
==========================================
+ Hits        11011    11028      +17     
  Misses      10020    10020              
+ Partials     1358     1357       -1

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

@alexgarel
docs: fixes after review in release
fa7d568 
+ set PERL5LIB in prod as we set environment
@alexgarel
Copy link
Member Author

alexgarel commented Jan 8, 2025 •
edited
Loading

@stephanegigandet applied proposed changes, and I also set the PERL5LIB path in setenv to make it easier !

stephanegigandet
stephanegigandet approved these changes Jan 8, 2025
View reviewed changes
Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@alexgarel alexgarel enabled auto-merge (squash) January 9, 2025 16:53
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Jan 9, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@alexgarel alexgarel merged commit d996a33 into main Jan 9, 2025
16 checks passed
@alexgarel alexgarel deleted the chore-deploy-process branch January 9, 2025 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephanegigandet stephanegigandet stephanegigandet approved these changes

Assignees

@alexgarel alexgarel

Labels
📚 Documentation Documentation issues improve the project for everyone.
Projects
📚 Document Open Food Facts
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexgarel @codecov-commenter @stephanegigandet