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

Use global npm registry #219

Closed
wants to merge 2 commits into from
Closed

Use global npm registry #219

wants to merge 2 commits into from

Conversation

Jakub-Kwak
Copy link
Contributor

Description

Describe the changes and motivations for the pull request, unless evident from the title.

How to test

  • FIXME: Add the steps describing how to verify your changes

Development checks

  • Add changeset according to guidelines (if needed)
PR commands

List of available commands:

  • @toptal-anvil ping reviewers - Ping FX team for review

@Jakub-Kwak Jakub-Kwak self-assigned this Oct 18, 2023
@Jakub-Kwak Jakub-Kwak requested a review from a team October 18, 2023 08:38
@changeset-bot
Copy link

changeset-bot bot commented Oct 18, 2023

⚠️ No Changeset found

Latest commit: 939cc18

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@github-actions github-actions bot added the contribution DO NOT ADD MANUALLY label Oct 18, 2023
@sashuk
Copy link
Collaborator

sashuk commented Oct 23, 2023

Hello, @Jakub-Kwak! Can you please specify if the pull request is ready for review? If not, can it be moved to draft for now?

@sashuk sashuk marked this pull request as draft October 23, 2023 06:33
@toptal toptal deleted a comment from toptal-devbot Oct 23, 2023
@Jakub-Kwak
Copy link
Contributor Author

@sashuk I've checked and this fix is still used only in screening-wizard-frontend. Let me ask the owner of the repo if they can switch back to master. If they confirm, I'll close this PR since it was only a temporary fix.

@Jakub-Kwak
Copy link
Contributor Author

Jakub-Kwak commented Oct 23, 2023

PR in screening-wizard-frontend created ☝️

@Jakub-Kwak
Copy link
Contributor Author

No workflow uses ci-fix-yarn-install branch anymore.

@Jakub-Kwak Jakub-Kwak closed this Oct 23, 2023
@Jakub-Kwak Jakub-Kwak deleted the ci-fix-yarn-install branch October 23, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants