Skip to content

Release 6.9.7#1638

Merged
vegaro merged 2 commits into
release/6.9.6from
release/6.9.7
Mar 7, 2024
Merged

Release 6.9.7#1638
vegaro merged 2 commits into
release/6.9.6from
release/6.9.7

Conversation

@vegaro

@vegaro vegaro commented Mar 7, 2024

Copy link
Copy Markdown
Member

Bugfixes

@vegaro vegaro added the pr:next_release Preparing a new release label Mar 7, 2024
@vegaro vegaro requested a review from a team March 7, 2024 15:13

@tonidero tonidero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a concern... but maybe we can fix the file after releasing this?

Comment thread .circleci/config.yml Outdated
command: aws s3 sync ~/project/docs/6.9.7 s3://purchases-docs/android/6.9.7 --delete
- run:
name: Update index.html
command: aws s3 cp ~/project/docs/index.html s3://purchases-docs/android/index.html

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hmm I just noticed but would this make the 6.9.7 docs the default ones? If so, we would need to either fix it so we don't overwrite the file if the versions is older than the one already there, or overwrite this file after publishing this

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ouch, yes, that's actually a bit complex to automate 🤔

fyi, I had to update the version number manually, because the lane tries to get the difference agains the latest version published, not the latest version within the same major, we need to fix that too...

Fine if I remove that step for now?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yup! We can remove that for now 👍

@codecov

codecov Bot commented Mar 7, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.93%. Comparing base (ce67866) to head (d1938db).

Additional details and impacted files
@@              Coverage Diff               @@
##           release/6.9.6    #1638   +/-   ##
==============================================
  Coverage          85.93%   85.93%           
==============================================
  Files                184      184           
  Lines               6277     6277           
  Branches             911      911           
==============================================
  Hits                5394     5394           
  Misses               536      536           
  Partials             347      347           

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

@vegaro vegaro merged this pull request into release/6.9.6 Mar 7, 2024
@vegaro vegaro deleted the release/6.9.7 branch March 7, 2024 17:52
@vegaro vegaro mentioned this pull request Mar 7, 2024
vegaro added a commit that referenced this pull request Mar 7, 2024
Adding #1638 to changelog in main
@JayShortway JayShortway restored the release/6.9.7 branch June 5, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:next_release Preparing a new release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants