Skip to content

🩹 [Patch]: Document usage of the Script parameter #28

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 24 commits into from
Jan 22, 2025

Conversation

MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Jan 22, 2025

Description

This pull request includes significant updates to the README.md file for improved clarity and additional examples, as well as changes to the TestWorkflow.yml to test various script paths.

Documentation improvements:

  • README.md: Improved descriptions, added clarity to existing examples, and included new examples for running scripts in different formats and contexts. [1] [2] [3] [4] [5] [6]

Workflow updates:

  • .github/workflows/TestWorkflow.yml: Added multiple test cases for running GitHub PowerShell scripts using different path syntaxes to ensure compatibility across various environments.

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@MariusStorhaug MariusStorhaug self-assigned this Jan 22, 2025
@MariusStorhaug MariusStorhaug changed the title 🩹 [Patch] 🚀[Feature]: Allow the Script parameter to also take a filepath (without dot-sourcing) Jan 22, 2025
@MariusStorhaug MariusStorhaug changed the title 🚀[Feature]: Allow the Script parameter to also take a filepath (without dot-sourcing) 🩹 [Patch]: Document usage of the Script parameter Jan 22, 2025
…yml and update README.md with supported script formats
…e the importance of using the correct action path variable
…ify script input formats for better understanding
@MariusStorhaug MariusStorhaug merged commit ca9e2ea into main Jan 22, 2025
36 checks passed
@MariusStorhaug MariusStorhaug deleted the notdotsource branch January 22, 2025 11:59
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.

🚀[Feature]: Allow the Script parameter to also take a filepath (without dot-sourcing)
1 participant