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

Migrates from 47deg to 47degrees GH organization #370

Merged
merged 4 commits into from Mar 15, 2020

Conversation

juanpedromoreno
Copy link
Member

No description provided.

@@ -70,7 +70,7 @@ You can list the users starring a specific repository with `listStargazers`; it
- `timeline`: whether or not to return the date at which point the user starred the repository.
- `pagination`: Limit and Offset for pagination, optional.

To list the stargazers of 47deg/github4s:
To list the stargazers of 47degrees/github4s:

```scala mdoc:compile-only
val listStargazers = Github[IO](accessToken).activities.listStargazers("47deg", "github4s", true)
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need to modify the examples too?

@BenFradet
Copy link
Contributor

it might be that the token needs updating?

@juanpedromoreno
Copy link
Member Author

The token was updated, the problem is that the test data was pointing to the old organization. Now it's fixed and ready for review :)

@codecov-io
Copy link

codecov-io commented Mar 15, 2020

Codecov Report

Merging #370 into master will increase coverage by 6.54%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #370      +/-   ##
==========================================
+ Coverage   86.16%   92.71%   +6.54%     
==========================================
  Files          23       23              
  Lines         535      535              
  Branches        1        1              
==========================================
+ Hits          461      496      +35     
+ Misses         74       39      -35     
Impacted Files Coverage Δ
github4s/src/main/scala/github4s/Decoders.scala 90.90% <0.00%> (+6.81%) ⬆️
...ub4s/src/main/scala/github4s/http/HttpClient.scala 93.75% <0.00%> (+25.00%) ⬆️
...4s/src/main/scala/github4s/http/Http4sSyntax.scala 100.00% <0.00%> (+33.33%) ⬆️
.../scala/github4s/interpreters/AuthInterpreter.scala 100.00% <0.00%> (+53.33%) ⬆️
.../src/main/scala/github4s/http/RequestBuilder.scala 100.00% <0.00%> (+60.00%) ⬆️
...s/src/main/scala/github4s/domain/PullRequest.scala 100.00% <0.00%> (+100.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04e36ef...4a46a12. Read the comment docs.

@juanpedromoreno juanpedromoreno merged commit 25844d5 into master Mar 15, 2020
@juanpedromoreno juanpedromoreno deleted the migrates-organization branch March 15, 2020 16:52
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

Successfully merging this pull request may close these issues.

None yet

4 participants