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

Make create-release-shortcut more resilient #6324

Merged
merged 5 commits into from
Feb 11, 2023
Merged

Conversation

fregante
Copy link
Member

@fregante fregante commented Feb 10, 2023

It seems that all shortcuts (even native ones) are broken at the moment 🤷‍♂️ but it should fix the first part of:

Test URLs

https://github.com/refined-github/refined-github/releases
https://github.com/refined-github/sandbox/releases/new
https://github.com/refined-github/sandbox/releases/tag/cool

Screenshot

Unable to test since it's natively broken

@fregante fregante added the bug label Feb 10, 2023
@fregante
Copy link
Member Author

c is already in use, dropping the feature instead:

Screenshot 5

Screenshot 4

I'm not sure why this feature was added since creating releases is such a relatively rare action 🤷‍♂️

@fregante fregante changed the title Make create-release-shortcut more resilient Drop create-release-shortcut feature Feb 11, 2023
@@ -3,7 +3,7 @@ import * as pageDetect from 'github-url-detection';

import features from '../feature-manager';

export default function addQuickSubmit(): void {
function addQuickSubmit(): void {
Copy link
Member Author

Choose a reason for hiding this comment

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

The fun part was that this part of the feature was being exported from submission-via-ctrl-enter-everywhere, duplicating it entirely

@fregante fregante changed the title Drop create-release-shortcut feature Make create-release-shortcut more resilient Feb 11, 2023
@fregante fregante merged commit da05ce3 into main Feb 11, 2023
@fregante fregante deleted the create-release-shortcut branch February 11, 2023 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

1 participant