Skip to content

Add 'infra' actions #5

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
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 gdemonet force-pushed the feature/add-infra-actions branch from 6b0407a to 515d53a Compare December 2, 2022 15:56
@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
@gdemonet gdemonet force-pushed the feature/add-infra-actions branch from a8f3b4b to 2905d9d Compare December 9, 2022 13:21
@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.

4 participants

@gdemonet @tcarmet @lavigne958