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.

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

Rewrite dist => lib in serverless.ymls #387

Merged
merged 3 commits into from
Mar 14, 2021
Merged

Conversation

72636c
Copy link
Member

@72636c 72636c commented Mar 14, 2021

skuba's default configuration compiles to lib, though it's common to see dist and lib out in the wild. skuba configure already tries to rewire Dockerfiles to point to this output directory, and now we do the same for Serverless files.

This implements globbing in our configure modules so we can match on multiple serverless*.ymls in the one repo.

skuba's default configuration compiles to `lib`, though it's common to
see `dist` and `lib` out in the wild. `skuba configure` already tries to
rewire Dockerfiles to point to this output directory, and now we do the
same for Serverless files.

This implements globbing in our configure modules so we can match on
multiple `serverless*.yml`s in the one repo.
@72636c 72636c requested a review from a team as a code owner March 14, 2021 10:27
@changeset-bot
Copy link

changeset-bot bot commented Mar 14, 2021

🦋 Changeset detected

Latest commit: 3610f6d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
skuba Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@72636c 72636c marked this pull request as draft March 14, 2021 10:33
@72636c 72636c marked this pull request as ready for review March 14, 2021 11:00
@72636c 72636c merged commit 8f17be6 into master Mar 14, 2021
@72636c 72636c deleted the serverless-configure-glob branch March 14, 2021 20:36
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.

None yet

2 participants