Skip to content

🌟 [Major]: Module folder restructure to support variables #41

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 21 commits into from
Aug 19, 2024

Conversation

MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Aug 16, 2024

Description

  • Update flow diagram.
  • Restructure expected module folder structure.
    • variables established under variables with a separate private and public folders.
    • To comply to the same structure, adjusted classes and functions to the same structure.
    • items under the public folders will be exported, the others are kept in the module scope.
  • Swap to use GitHub-Script@v1, instead of pure inline scripts.

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 requested a review from a team as a code owner August 16, 2024 21:17
@MariusStorhaug MariusStorhaug changed the title VariableToExport 🌟 [Major]: Module folder restructure to support variables Aug 19, 2024
@MariusStorhaug MariusStorhaug self-assigned this Aug 19, 2024
@MariusStorhaug MariusStorhaug merged commit b3d380e into main Aug 19, 2024
8 checks passed
@MariusStorhaug MariusStorhaug deleted the VariablesToExport branch August 19, 2024 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant