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

fix: switch lib to parse yaml #414

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

crazy-max
Copy link
Member

fixes #397

Switch to yaml lib to properly parse goreleaser config.

@crazy-max crazy-max requested a review from caarlos0 as a code owner May 7, 2023 15:26
@codecov
Copy link

codecov bot commented May 7, 2023

Codecov Report

Merging #414 (0d8dcf3) into master (3724734) will increase coverage by 0.50%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master     #414      +/-   ##
==========================================
+ Coverage   67.18%   67.69%   +0.50%     
==========================================
  Files           3        3              
  Lines         128      130       +2     
  Branches       22       23       +1     
==========================================
+ Hits           86       88       +2     
+ Misses         27       26       -1     
- Partials       15       16       +1     
Impacted Files Coverage Δ
src/goreleaser.ts 56.33% <60.00%> (+1.26%) ⬆️

@crazy-max crazy-max marked this pull request as draft May 7, 2023 15:40
@crazy-max crazy-max force-pushed the fix-yaml-parsing branch from b37c79a to 0d8dcf3 Compare May 7, 2023 15:43
@crazy-max crazy-max marked this pull request as ready for review May 7, 2023 15:45
@crazy-max crazy-max marked this pull request as draft May 7, 2023 16:01
@caarlos0
Copy link
Member

i wonder if it isn't simpler to let the user set the dist folder in the action too, and not even bother parsing any yaml 🙇‍♂️

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.

YAML parse error when .goreleaser.yml contains anchor
2 participants