Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Chore: rm automatic NPM publising #180

Merged
merged 1 commit into from
Dec 21, 2021
Merged

Chore: rm automatic NPM publising #180

merged 1 commit into from
Dec 21, 2021

Conversation

katspaugh
Copy link
Member

Publishing to NPM w/o 2FA is a security concern, so we're removing this.

@katspaugh katspaugh requested a review from DaniSomoza December 21, 2021 10:16
DaniSomoza
DaniSomoza approved these changes Dec 21, 2021
View reviewed changes
Copy link
Contributor

@DaniSomoza DaniSomoza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved! :D

@github-actions
Copy link

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total
Report generated by eslint-plus-action

@github-actions
Copy link

📚 Storybook review

@katspaugh katspaugh merged commit f8037c0 into main Dec 21, 2021
@katspaugh katspaugh deleted the rm-npm branch December 21, 2021 10:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DaniSomoza DaniSomoza DaniSomoza approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@katspaugh @DaniSomoza