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

Bugfix/macos release overrides #112

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

iatanasov37
Copy link

@iatanasov37 iatanasov37 commented Sep 8, 2023

This Pull request fixes an issue, where the action fails if running on the currently latest official MacOS version due to an outdated dependency.

This is related to issue: #113

Fix has been tested by using the following flow:

  1. npm ci
  2. npm run pack
  3. node dist/index.js

Before the fix the above flow was generating the same error as described in the issue. After the fix, we are getting errors since no input was actually passed down when running the index.

Tested on:
Chip: Apple M1
MacOS: Ventura 13.5.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant