Skip to content

Checks for relative path names #15

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

Merged
merged 6 commits into from
Aug 25, 2019
Merged

Checks for relative path names #15

merged 6 commits into from
Aug 25, 2019

Conversation

JamesIves
Copy link
Owner

@JamesIves JamesIves commented Aug 25, 2019

Description

Provide a description of what your changes do.

Makes a check to see if ./ or / is included at the start of the $FOLDER variable. This will prevent an assertion error being thrown. Details: #14

Testing Instructions

Give us step by step instructions on how to test your changes.

  • Try deploying when $FOLDER starts with ./ or /, it will error and cancel the script. For example build/thing will work, but ./build/thing or /build/thing doesn't work.

Additional Notes

Anything else that will help us test the pull request.

Also added the action.yml file for meta data and updated the read me.

Closes #14

@JamesIves JamesIves merged commit f7faf64 into master Aug 25, 2019
@JamesIves JamesIves deleted the relative branch August 25, 2019 22:16
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.

Branch deleted after "deploy"
1 participant