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

Use environment in make-release.yml #7782

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Conversation

nirinchev
Copy link
Member

What, How & Why?

☑️ ToDos

  • 📝 Changelog update
  • 🚦 Tests (or not relevant)
  • C-API, if public C++ API changed
  • bindgen/spec.yml, if public C++ API changed

@nirinchev nirinchev requested a review from ironage June 6, 2024 06:28
@cla-bot cla-bot bot added the cla: yes label Jun 6, 2024
@nirinchev nirinchev added no-changelog no-jira-ticket Skip checking the PR title for Jira reference labels Jun 6, 2024
Copy link

coveralls-official bot commented Jun 6, 2024

Pull Request Test Coverage Report for Build nikola.irinchev_477

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 27 unchanged lines in 6 files lost coverage.
  • Overall coverage increased (+0.009%) to 90.867%

Files with Coverage Reduction New Missed Lines %
src/realm/sync/noinst/server/server.cpp 1 73.67%
src/realm/util/serializer.cpp 1 90.43%
src/realm/util/assert.hpp 4 87.1%
src/realm/sync/transform.cpp 6 60.73%
src/realm/bplustree.cpp 7 72.39%
src/realm/index_string.cpp 8 84.63%
Totals Coverage Status
Change from base Build 2388: 0.009%
Covered Lines: 214831
Relevant Lines: 236424

💛 - Coveralls

@@ -12,6 +12,9 @@ jobs:
finish_release:
if: github.event.pull_request.merged == true && startsWith(github.head_ref, 'release/automated_v')
runs-on: ubuntu-latest
environment:
name: Production
url: ${{ steps.publish-github-release.outputs.html_url }}
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if this variable will be available at the beginning of the run, or if it is evaluated after the run... we shall see :)

Copy link
Member Author

Choose a reason for hiding this comment

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

This is evaluated after the run

@nirinchev nirinchev merged commit 06b658b into master Jun 6, 2024
42 of 48 checks passed
@nirinchev nirinchev deleted the ni/release-environment branch June 6, 2024 23:23
@github-actions github-actions bot mentioned this pull request Jun 7, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes no-changelog no-jira-ticket Skip checking the PR title for Jira reference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants