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

Add short uri #25

Open
afragen opened this issue Nov 2, 2013 · 10 comments
Open

Add short uri #25

afragen opened this issue Nov 2, 2013 · 10 comments

Comments

@afragen
Copy link
Contributor

afragen commented Nov 2, 2013

I'll have to figure out how to add the <owner>/<repo> as an option for updating. To effect this the base URI would be placed from the header i.e.,

GitHub Plugin URI => https://github.com/
BitBucket Plugin URI => https://bitbucket.com/
etc.

@pdclark
Copy link
Owner

pdclark commented Nov 2, 2013

Is there an advantage to doing it this way when all the info can be detected by copying-and-pasting the repo URL?

I assume you're talking about a format like this?
GitHub URI: brainstormmedia/git-plugin-updates

Is this for compatibility with something that's already out there?

@afragen
Copy link
Contributor Author

afragen commented Nov 2, 2013

Well github-updater supports it 😉

It was a feature request from @GaryJones

@afragen
Copy link
Contributor Author

afragen commented Nov 2, 2013

The BIG advantage I can see is so that any .org hosted plugin whose development is on GitHub or BitBucket doesn't get flagged for updating because of the plugin. WP.org repo plugins should only get an update instruction for .org

@pdclark
Copy link
Owner

pdclark commented Nov 2, 2013

This is still confusing to me. Maybe we could chat on Skype?

If the purpose is to provide two ways to configure the updater, it seems redundant.

If the purpose it to display a link to Git without firing updates, I think that's better done in a GUI and left up to the user — the plugin's only going to update if the readme version is bumped anyway.

@pdclark
Copy link
Owner

pdclark commented Nov 2, 2013

Or put another way, what is the problem this solves?

/*
Plugin URI: http://wordpress.org/someplugin
Git URI: http://github.com/owner/someplugin
*/

/*
Plugin URI: http://wordpress.org/someplugin
Git(hub) URI: owner/someplugin
*/

These both do the same thing, except that now a user can't copy-and-paste a link, and the definition of URI is being ignored.

@afragen
Copy link
Contributor Author

afragen commented Nov 2, 2013

I'm on call today so Skype won't work but here's where it causes an issue.

Plugin URI: https://github.com/afragen/events-calendar-category-colors

This is what I have had as the Plugin URI for a at least 2 years. As a test I decreased the version number an this plugin updated it, not the repo.

@pdclark
Copy link
Owner

pdclark commented Nov 3, 2013

Plugin URI should be fixed (disabled) in this commit: 20fb1ac

Can you tell me a bit more about supporting owner/repo in the Git URI field? Is it just for shorthand?

@pdclark
Copy link
Owner

pdclark commented Nov 3, 2013

(Or, just link me to @GaryJones's original feature request.)

@GaryJones
Copy link

The gist of it was, we had something like:

GitHub URL: **https://github.com/**.../...

The domain was redundant, since we'd already specified "GitHub" as the plugin header.

@pdclark
Copy link
Owner

pdclark commented Nov 4, 2013

If you want to submit a pull request to allow that with the legacy headers, I'll accept it. As long as we don't document it as an option.

It'd also be fine for Git URI: to assume Github if no domain, but again, would rather not document that it allows for that. I think providing two options to config to do the same thing adds confusion (and it's not what a URI is!).

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

No branches or pull requests

3 participants