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

Support scoped modules. #48

Closed
wants to merge 1 commit into from
Closed

Support scoped modules. #48

wants to merge 1 commit into from

Conversation

hemanth
Copy link
Contributor

@hemanth hemanth commented Oct 6, 2015

^

@bahmutov
Copy link

bahmutov commented Dec 3, 2015

+1 on this one

@hemanth
Copy link
Contributor Author

hemanth commented Dec 3, 2015

/me paws @gorillamania

@bahmutov
Copy link

bahmutov commented Dec 3, 2015

I switched to using @hemanth fork in my https://github.com/bahmutov/grunt-nice-package for now

@@ -62,7 +62,8 @@ QUnit.test("Dependencies Ranges", function() {
'tilde-first': '~1.2',
'x-version': '1.2.x',
'tilde-top': '~1',
'caret-top': '^1'
'caret-top': '^1',
"@es-shims/api": "^1.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

It would have been nice to follow the existing style of single quotes here...

Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
"@es-shims/api": "^1.0.0"
'@es-shims/api': '^1.0.0'

@LinusU
Copy link
Collaborator

LinusU commented Sep 22, 2016

ping @gorillamania would really appreciate getting this merged. I'd be happy to step up as a maintainer if you no longer have time to devote to this project :)

@TechNickAI
Copy link
Collaborator

Yes please! Note this has conflicts that prevent it from being merged. What is your npm username?I'll add you as a maintainer

@LinusU
Copy link
Collaborator

LinusU commented Sep 24, 2016

Glad to hear! I'm linusu on npm :)

@TechNickAI
Copy link
Collaborator

@LinusU I've added you as a collaborator on this repo and on NPM. Really appreciate the help!

@hemanth
Copy link
Contributor Author

hemanth commented Sep 27, 2016

Meow.

Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

🚀 Looks great! Just the one style point on keeping to the existing ' for the added line.

@JoshuaKGoldberg
Copy link
Owner

@hemanth I don't have permissions to push the suggestion (or merge conflict resolutions) to your branch. Would you be up for either doing that yourself or giving me permission to?

I know it's been a long time, but I'd love to get this in 😄

@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author Needs an action taken by the original poster label Mar 28, 2024
@hemanth
Copy link
Contributor Author

hemanth commented Mar 28, 2024

@JoshuaKGoldberg I guess @bahmutov has a better fix :)

@JoshuaKGoldberg
Copy link
Owner

Sorry, I don't follow - what's the better fix?

@hemanth
Copy link
Contributor Author

hemanth commented Mar 28, 2024

#48 (comment)

Looks like my fork is deleted, so ^ :|

@JoshuaKGoldberg
Copy link
Owner

Ah, gotcha.

For context: I want to accept these changes and give you credit as the author. Do you want to re-fork and send a new PR?

If not / you don't have time, no worries, I can always make them myself and list you as a co-author. But I'd feel bad doing that without giving you the opportunity to be the primary author. 🙂

@hemanth
Copy link
Contributor Author

hemanth commented Mar 28, 2024

@JoshuaKGoldberg That's ok, FOSS FTW! ❤️

@JoshuaKGoldberg JoshuaKGoldberg deleted the branch JoshuaKGoldberg:master July 8, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting for author Needs an action taken by the original poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants