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.

Sign up for GitHub

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

Jump to bottom

Added version endpoint #4

Merged
merged 2 commits into from
May 31, 2021
Merged

Added version endpoint #4

merged 2 commits into from
May 31, 2021

Conversation

applejag
Copy link
Contributor

@applejag applejag commented May 11, 2021 •
edited
Loading

  • Added version.go with GET /import/github/version endpoint
  • Added deploy/update-version.sh to update version.yaml on Docker build
  • Updated main.go with metadata
  • Updated .wharf-ci.yml to populate BUILD_* build args
  • Updated CHANGELOG.md
  • Updated swag init docs & usage as we're now relying on types from foreign packages (app.Version)

Based on iver-wharf/wharf-core#2

Screenshot_2021-05-28_16-33-25

Closes #5

@applejag applejag added the enhancement New feature or request label May 11, 2021
@applejag applejag self-assigned this May 11, 2021
@applejag applejag added this to In progress in Backlog via automation May 11, 2021
@applejag applejag force-pushed the feature/version-endpoint branch 2 times, most recently from fa305a6 to dda96c4 Compare May 11, 2021 13:59
@applejag applejag mentioned this pull request May 11, 2021
Merged
@applejag applejag moved this from In progress to Review in progress in Backlog May 11, 2021
@applejag applejag mentioned this pull request May 11, 2021
Merged
@applejag applejag force-pushed the feature/version-endpoint branch 2 times, most recently from 2f9fe3c to 2b640ad Compare May 11, 2021 14:36
Pikabanga
Pikabanga reviewed May 11, 2021
View reviewed changes
Copy link

@Pikabanga Pikabanga left a comment

Choose a reason for hiding this comment

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

Same comments as in iver-wharf/wharf-provider-github#5

Pikabanga
Pikabanga suggested changes May 12, 2021
View reviewed changes
Copy link

@Pikabanga Pikabanga left a comment

Choose a reason for hiding this comment

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

Flagging for requesting changes

@applejag applejag marked this pull request as draft May 12, 2021 09:29
@applejag applejag marked this pull request as ready for review May 28, 2021 14:32
@applejag applejag requested a review from Pikabanga May 28, 2021 14:34
Backlog automation moved this from Review in progress to Reviewer approved May 31, 2021
@applejag applejag merged commit 305e27c into master May 31, 2021
Backlog automation moved this from Reviewer approved to Done May 31, 2021
@applejag applejag deleted the feature/version-endpoint branch May 31, 2021 08:33
@applejag applejag mentioned this pull request Jul 5, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fredx30 fredx30 fredx30 approved these changes

@Pikabanga Pikabanga Pikabanga approved these changes

@iverestefans iverestefans Awaiting requested review from iverestefans

Assignees

@applejag applejag

Labels
enhancement New feature or request
Projects
No open projects
Backlog
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Version endpoint
3 participants
@applejag @fredx30 @Pikabanga