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

replace docker publish action ::set-output usage #4359

Merged
merged 1 commit into from Jul 18, 2023
Merged

replace docker publish action ::set-output usage #4359

merged 1 commit into from Jul 18, 2023

Conversation

kenjenkins
Copy link
Contributor

Summary

Update the 'Docker Main' GitHub Action workflow to update the deprecated ::set-output command to use the newer environment file mechanism.

Related issues

#4358

User Explanation

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@kenjenkins
replace docker publish action ::set-output usage
bda3dd8 
Update the 'Docker Main' GitHub Action workflow to update the deprecated
::set-output command to use the newer environment file mechanism.
@kenjenkins kenjenkins requested a review from a team as a code owner July 12, 2023 17:42
@kenjenkins kenjenkins requested a review from wasaga July 12, 2023 17:42
@coveralls
Copy link

Coverage Status

coverage: 63.484% (+0.06%) from 63.423% when pulling bda3dd8 on kenjenkins/docker-set-output into 4674b98 on main.

@kenjenkins kenjenkins merged commit 5c0d7aa into main Jul 18, 2023
10 checks passed
@kenjenkins kenjenkins deleted the kenjenkins/docker-set-output branch July 18, 2023 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calebdoxsey calebdoxsey calebdoxsey approved these changes

@wasaga wasaga Awaiting requested review from wasaga wasaga is a code owner automatically assigned from pomerium/dev-backend

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kenjenkins @coveralls @calebdoxsey