Skip to content
This repository was archived by the owner on Sep 15, 2022. It is now read-only.

[WIP] refactoring/new extension#100

Open
PedroTroller wants to merge 33 commits intomasterfrom
refactoring/new-extension
Open

[WIP] refactoring/new extension#100
PedroTroller wants to merge 33 commits intomasterfrom
refactoring/new-extension

Conversation

@PedroTroller
Copy link
Copy Markdown
Contributor

No description provided.

@PedroTroller PedroTroller force-pushed the refactoring/new-extension branch from 9fc5805 to 6bc0c04 Compare October 24, 2014 12:54
devnull Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Youps ...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As any dependency, the minimum stability should be dev.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 . An other fact is that you restrict the entire require and require-dev section to be stable too. This not really needed and more this is not flexible ^^.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's a dev restriction ;) It will stay like that until the release.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I am not sure the password should be here

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

indeed :') . Notice that you should change your password before somebody else do it...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't care, it's an email address dedicated to tests.
BTW, I've activated 2 steps connexion.

@einenlum
Copy link
Copy Markdown

@Nek- @PedroTroller Shouldn't we close this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants