Skip to content

feat(internal): auto-run godocfx on new mods#3069

Merged
gcf-merge-on-green[bot] merged 3 commits intomasterfrom
auto-godocfx
Oct 22, 2020
Merged

feat(internal): auto-run godocfx on new mods#3069
gcf-merge-on-green[bot] merged 3 commits intomasterfrom
auto-godocfx

Conversation

@tbpg
Copy link
Copy Markdown
Contributor

@tbpg tbpg commented Oct 22, 2020

This uses index.golang.org to list all modules since a given time. For
example, see
https://index.golang.org/index?since=2019-04-10T19:08:52.997264Z. Each
page is listed in chronological order and is limited to 2000 entries (as
of today).

This stores the last successful time in Datastore. If there is no time in
Datastore, it defaults to 10 days ago.

There is a bit of indirection with interfaces to enable testing.

In the future, we should add the ability to regenerate the YAML for the
latest version of all modules.

cc @julieqiu @katiehockman

This uses index.golang.org to list all modules since a given time. For
example, see
https://index.golang.org/index?since=2019-04-10T19:08:52.997264Z. Each
page is listed in chronological order and is limited to 2000 entries (as
of today).

This stores the last successful time in Datastore. If there is no time in
Datastore, it defaults to 10 days ago.

There is a bit of indirection with interfaces to enable testing.

In the future, we should add the ability to regenerate the YAML for the
latest version of all modules.
@tbpg tbpg requested review from a team and codyoss October 22, 2020 14:39
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 22, 2020
Copy link
Copy Markdown
Member

@codyoss codyoss left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread internal/godocfx/timesaver.go
Copy link
Copy Markdown
Contributor Author

@tbpg tbpg left a comment

Choose a reason for hiding this comment

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

Thanks!

Comment thread internal/godocfx/timesaver.go
@tbpg tbpg added the automerge Merge the pull request once unit tests and other checks pass. label Oct 22, 2020
@gcf-merge-on-green gcf-merge-on-green Bot merged commit 49f497e into master Oct 22, 2020
@gcf-merge-on-green gcf-merge-on-green Bot deleted the auto-godocfx branch October 22, 2020 15:28
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants