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

Patch Singularity GS #101

Open
wants to merge 6 commits into
base: update-gulp-sass-for-yarn
Choose a base branch
from
Open

Conversation

byrond
Copy link

@byrond byrond commented Oct 30, 2018

After updating gulp-sass on #100, several DEPRECATED warnings are issued when building the styleguide. Singularity is EOL and no longer maintained, so it must be patched (see https://github.com/at-import/Singularity/releases/tag/v1.8.0).

This PR updates Butler using the method in at-import/Singularity#245 (comment)

After updating or creating the styleguide for a project, run:

  • yarn patch-package

This will apply the patch to your instance (it isn't automatically run when adding Butler). The installation instructions in Butler's readme will need to be updated accordingly. However, they are currently written for using npm. The patch requires Yarn, because the path to Singularity doesn't include node_modules/butler/node_modules.

This PR also includes a limited update to npm-shrinkwrap.json that only changes the URL of the 'asap' package, removing the internal Uber path. It replaces #90, since there were concerns there about regressions.

@byrond
Copy link
Author

byrond commented Oct 30, 2018

Adding "prepare": "patch-package" to the scripts section of package.json will apply patches during yarn install. That can also be added to the readme.

@becw
Copy link
Member

becw commented Jan 9, 2019

Would it be possible to document where this patch comes from for future code archaeologists? Perhaps a PATCHES-butler.txt file?

@becw
Copy link
Member

becw commented Jan 9, 2019

@byrond sorry to even ping you on this! We aren't starting new projects with Butler these days, so I'm not worried about getting this PR wrapped up -- in fact I probably won't follow up on it at all!

@byrond
Copy link
Author

byrond commented Jan 9, 2019

No worries, @becw! The patch is from a commit to a repository that referenced that Singularity GS issue. jobiqo/recruiter_epiq_deps@6bf87a1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants