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

Add status to outputs #147

Merged
merged 3 commits into from
May 4, 2024
Merged

Add status to outputs #147

merged 3 commits into from
May 4, 2024

Conversation

jerturowetz
Copy link

@jerturowetz jerturowetz commented Apr 24, 2024

Closes #146

Hopefully this is the only thing I'd need to change. Not super familiar with Axios here :P

@jerturowetz jerturowetz changed the title Add status to Axios outputs Add status to outputs Apr 24, 2024
@fjogeleit
Copy link
Owner

fjogeleit commented Apr 25, 2024

You need to build the action, otherwise your change is not active.

npm run build

also please add the output to the readme response section

@jerturowetz
Copy link
Author

jerturowetz commented May 3, 2024

You need to build the action, otherwise your change is not active.

npm run build

also please add the output to the readme response section

@fjogeleit Thanks for the guidance!

@fjogeleit fjogeleit merged commit 7d4a18a into fjogeleit:main May 4, 2024
3 checks passed
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.

Any way to retrieve the response status code?
3 participants