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.

Sign up for GitHub

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

Jump to bottom

Add 'infra' actions #5

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Add 'infra' actions #5

wants to merge 3 commits into from

Conversation

gdemonet
Copy link

@gdemonet gdemonet commented Dec 2, 2022 •
edited
Loading

⚠️ This is still a work in progress, do not hesitate to comment and discuss

Goals:

  • setup a single npm root package with multiple workspaces for generating a full hierarchy of actions
    • add statically generated docs website (with client-side embedded search)
  • ensure all generated actions are up-to-date with the Typescript sources, properly linted and tested
  • implement a set of actions for provisioning (and cleaning) cloud infrastructure (either with Terraform or Pulumi, depending on the initial tests with Pulumi)

@gdemonet
Initialize top-level npm package
36314ff 
All the building and testing will be executable from the root of the
repository, for ease of use.
Logically separated actions will live under cross-dependent npm
workspaces (requires npm 7 or higher).
@gdemonet gdemonet marked this pull request as draft December 2, 2022 15:55
@gdemonet
Setup an 'infra' workspace and an 'infra/example' action
7331e27 
This simply demonstrates how we can generate many actions from a single
npm package.
Implementation of actual actions will follow.
@gdemonet gdemonet force-pushed the feature/add-infra-actions branch 24 times, most recently from eda9615 to a8f3b4b Compare December 9, 2022 13:02
@tcarmet
Copy link
Contributor

tcarmet commented Jan 2, 2023

I like what you're trying to do here, especially towards reorganizing the repo and the documentation part.

I haven't read the full PR in deep but I wonder if the infra part wouldn't deserve its own repository. I have no strong opinion at the moment, just trying to look what you think its best. If you don't know yet, we can always start here and then move the action to a new repo if needed.

@lavigne958 lavigne958 added bump_version Identify automatic PR that bump version after a release and removed bump_version Identify automatic PR that bump version after a release labels Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gdemonet @tcarmet @lavigne958