Update infrastructure - Githubissues

neomatrixcode / Diana.jl

GraphQL for Julia

https://diana.nicepage.io/

MIT License

115 stars 16 forks source link

Update infrastructure #8

Closed Nosferican closed 5 years ago

Nosferican commented 5 years ago

Community standards
- Template for bug repors, feature requests, and custom
Pkg for Julia 1
- Drop REQUIRE for Project
- Added Project for docs and test
Updated the Travis CI config file
Update LICENSE.md
- Use the default MIT format so it may be detected by Licensee (GitHub)
- Attributed copyright to legal name instead of GitHub handle
Changes to README
- Removed outdated PkgEval
- Moved roadmap to change log file
- Moved documentation to the docs infrastructure

Actions required,

Enable Documenter for the Travis CI
Add Registrator/Tag Bot
If the environmental variables are used for the API tokens, the examples could be run at the docs stage

Nosferican commented 5 years ago

The CI error seems to on test/cliente.jl ln:27

client = GraphQLClient("https://api.graph.cool/simple/v1/movies",auth="Bearer my-jwt-token",headers=Dict("header"=>"value"))

when I tried the tests locally it would give an error due to

Project has exceeded its limits: number of requests.

I have a version with the test_throws/test_broken, but it would be better to include a new key as an environmental variables in Travis CI. I am unsure if currently running the appveyor.yml or should it be deleted.

Nosferican commented 5 years ago

The docs are not being hosted at the moment since the build fails due to the api.graph.cool. We could skip/test_broken for now such that the documents are hosted and users can see the examples.

neomatrixcode commented 5 years ago

I will skip those tests, then I will raise a server for them to work