babel-plugin-inline-react-svg icon indicating copy to clipboard operation
babel-plugin-inline-react-svg copied to clipboard

Register export declaration in scope

Open forivall opened this issue 4 years ago • 4 comments

Fixes #74

See https://github.com/airbnb/babel-plugin-inline-react-svg/pull/77#issuecomment-931439820

@briganti this is your change, i'm just submitting it to hopefully get the ball rolling.

forivall avatar Nov 15 '21 22:11 forivall

@forivall note how https://github.com/airbnb/babel-plugin-inline-react-svg/pull/77#issuecomment-856833786 explicitly says "not a PR"? This doesn't get things moving, it creates a permanent stain on the repo, and the only way to mitigate that is for me to now maintain two PRs in sync - meaning, please never close this one or delete the branch or repo.

ljharb avatar Nov 15 '21 22:11 ljharb

Sorry, I missed that in the comment thread, just skimmed it, and when I used the compare tool in GitHub, it gave a tempting "create a pull request" button, so I clicked it. Sorry again.

forivall avatar Nov 16 '21 00:11 forivall

Hi @ljharb

Anything I can help with to get this or the one in the last comment of #77 get merged?

rdhlb avatar Nov 25 '21 13:11 rdhlb

@rdhlb no, at this point there's nothing that can be done to help; it'll just take time.

ljharb avatar Nov 25 '21 16:11 ljharb

@forivall can you please check the "allow edits" box on the RHS of the PR?

ljharb avatar Feb 22 '23 19:02 ljharb

Checkbox isn't there for me. @briganti, is it there for you? Or maybe you'd have to give me access to your repo if not? Screenshot_20230222-115331 https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

forivall avatar Feb 22 '23 19:02 forivall

ohhhh you opened a PR from briganti's repo - in that case nvm, i don't think this can be updated at all, so i'll just close it (i'd also never have been able to land it without briganti's cooperation anyways; please don't open PRs from repos that aren't yours)

ljharb avatar Feb 22 '23 20:02 ljharb