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

refactor: migrate off from set-output #297

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

kirillzyusko
Copy link
Owner

@kirillzyusko kirillzyusko commented Dec 14, 2023

📜 Description

Migrate off from set-output.

💡 Motivation and Context

This command is deprecated and will be disabled in near future, so it's better to migrate to the variant that will not produce this warning.

📢 Changelog

CI

  • used modern syntax instead of ::set-output;
  • synchronized code between e2e actions;

🤔 How Has This Been Tested?

Tested via GH actions.

📸 Screenshots (if appropriate):

image

📝 Checklist

  • CI successfully passed

@kirillzyusko kirillzyusko added CI CI (github actions) refactor You changed the code but it didn't affect functionality labels Dec 14, 2023
@kirillzyusko kirillzyusko self-assigned this Dec 14, 2023
Copy link
Contributor

github-actions bot commented Dec 14, 2023

📊 Package size report

Current size Target Size Difference
99245 bytes 99245 bytes 0 bytes 📉

@kirillzyusko kirillzyusko marked this pull request as ready for review December 14, 2023 21:39
@kirillzyusko kirillzyusko merged commit 5492356 into main Dec 15, 2023
10 checks passed
@kirillzyusko kirillzyusko deleted the refactor/deprecated-set-output branch December 15, 2023 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI (github actions) refactor You changed the code but it didn't affect functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant