-
Notifications
You must be signed in to change notification settings - Fork 6
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
Change auto deployment to use GitHub actions #127
Conversation
74b5b19
to
65ad787
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Changes LGTM, does it make sense to add a badge to show last deployment date for this branch? 👍
@clue, I think you're right, I'll add this to my PR 👍 |
65ad787
to
e26e838
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thanks for the update, changes LGTM!
e26e838
to
406aa5b
Compare
Also added a CI badge :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Badge URL looks outdated, no?
406aa5b
to
fcd335b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review the badge URL and use the current format.
fcd335b
to
3536233
Compare
@clue This should be the right one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonFrings Thanks for the update!
Refs reactphp/website#53 and clue/framework-x-website#7.
Builds on top of #125 and #126