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

fix: Rely on environment input instead of fixed matrices #9

Merged
merged 1 commit into from
Apr 13, 2023
Merged

fix: Rely on environment input instead of fixed matrices #9

merged 1 commit into from
Apr 13, 2023

Conversation

manuelcoppotelli
Copy link
Contributor

List of Changes

Rely on environment input instead of matrix to select the environment to use

Motivation and Context

Fixes #8

How Has This Been Tested?

Sandbox environment in a separate forked repo

Screenshots (if appropriate):

Types of changes

  • Chore (nothing changes by a user perspective)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@manuelcoppotelli manuelcoppotelli requested a review from a team April 8, 2023 10:15
@manuelcoppotelli manuelcoppotelli self-assigned this Apr 8, 2023
@manuelcoppotelli manuelcoppotelli requested a review from gunzip April 8, 2023 10:15
@manuelcoppotelli manuelcoppotelli changed the title fix: use environment input fix: Rely on environment input instead of fixed matric Apr 8, 2023
@manuelcoppotelli manuelcoppotelli changed the title fix: Rely on environment input instead of fixed matric fix: Rely on environment input instead of fixed matrices Apr 8, 2023
@pasqualedevita pasqualedevita merged commit fa12a51 into main Apr 13, 2023
@pasqualedevita pasqualedevita deleted the fix-environment-input-matrix branch April 13, 2023 09:09
@github-actions
Copy link

🎉 This PR is included in version 1.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pasqualedevita pasqualedevita pasqualedevita approved these changes

@gunzip gunzip gunzip approved these changes

Assignees

@manuelcoppotelli manuelcoppotelli

Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

action should be agnostic with respect to the matrix setup
3 participants
@manuelcoppotelli @gunzip @pasqualedevita