Skip to content
This repository has been archived by the owner on Apr 21, 2022. It is now read-only.

Update Readme explaining #48 #87

Merged
merged 4 commits into from
Mar 25, 2019
Merged

Conversation

MatejMecka
Copy link
Contributor

At First people will think there is an error when running it. So i'm sending this Pull Request in order to address the issue and that it's perfectly ok

This should close: #48

@renatomefi
Copy link

But how do we configure it to run our information in dev mode? I think this information is rather valuable!

Thanks

@MatejMecka
Copy link
Contributor Author

@renatomefi Can you check if it's ok?

@renatomefi
Copy link

@MatejMecka yes, great, thanks!

README.md Outdated Show resolved Hide resolved
@MatejMecka
Copy link
Contributor Author

@brandonrosage Can you review this PR so it can be merged?

Copy link
Collaborator

@brandonrosage brandonrosage left a comment

Choose a reason for hiding this comment

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

@MatejMecka I've revised the language a bit and linked to detailed documentation about using API authentication. Look okay?

@MatejMecka
Copy link
Contributor Author

Yes it looks perfect!

@brandonrosage brandonrosage merged commit c443419 into github:master Mar 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants