Skip to content

change http to https where applicable #293

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

Closed
wants to merge 1 commit into from
Closed

change http to https where applicable #293

wants to merge 1 commit into from

Conversation

OmgImAlexis
Copy link

https://xmlsoft.org/ has an incorrect SSL cert so I couldn't change that.
Sourceforge redirects https to http so I couldn't change that either.

@scottgonzalez
Copy link
Member

Please fix the whitespace.

@OmgImAlexis
Copy link
Author

@scottgonzalez this repo also needs a editorconfig. Nothing I can really do with my editor until one is added as it'll auto convert the tabs to spaces on save unless there's an editorconfig.

I'll just wait for it to be added before fixing it in this repo and others I've got PRs open on.

@jzaefferer
Copy link
Member

I'll just wait for it to be added before fixing it in this repo and others I've got PRs open on.

Or add a copy of https://github.com/jquery/api.qunitjs.com/blob/master/.editorconfig to this repo.

@OmgImAlexis OmgImAlexis mentioned this pull request Dec 10, 2015
@scottgonzalez
Copy link
Member

Ok, I'll wait for the editorconfig update to land first, but you should really look into your editor's settings if you truly believe there's nothing you can do without a config file. Either your editor is severely limited (which I doubt), or there are settings you're just not aware of.

@OmgImAlexis
Copy link
Author

It's not that, I'm using atom.io and I have a few hundred files I'm working on at the moment. Currently in OSX if you close ANY window all of them cause it to crash and editing the config will update all of my files. If I wait for the editorconfig to be added I won't have to manually save and close all of them just to change the settings.

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

Successfully merging this pull request may close these issues.

4 participants