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

Handle case where there is no user .yamlrc.yml file present #55

Closed

Conversation

dannyvv
Copy link
Member

@dannyvv dannyvv commented Dec 26, 2024

When adding support for .yarnrc files in a previous commit a use case was missed.
This duplicates the logic of .npmrc where the user profile .yarnrc file is initialized empty if not present or if present the actual default hard-coded feed registration is removed to ensure compliance with Microsoft Security (SFI) requirements

@dannyvv dannyvv closed this Dec 27, 2024
@dannyvv dannyvv force-pushed the user/dannyvv/pr/FixNoYamlRcInUser branch from 18e8461 to 933ed32 Compare December 27, 2024 17:43
@dannyvv
Copy link
Member Author

dannyvv commented Dec 27, 2024

Not sure why GH decided to close the PR when I accidentally pushed the code to a repo-branch :(
New PR: #56

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.

1 participant