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

Update app template #93

Merged
merged 12 commits into from
Feb 8, 2021
Merged

Update app template #93

merged 12 commits into from
Feb 8, 2021

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Jan 19, 2021

What does this PR do?

Update the application template:

  • bump dependencies
  • add github actions
  • add functional test framework

@Aschen Aschen changed the base branch from master to develop January 19, 2021 05:59
@Aschen Aschen self-assigned this Jan 19, 2021
@Aschen Aschen marked this pull request as draft January 19, 2021 11:13
@Aschen Aschen marked this pull request as ready for review January 29, 2021 11:12
@Aschen Aschen linked an issue Jan 29, 2021 that may be closed by this pull request
@Aschen Aschen force-pushed the update-app-template branch from 3e1abce to aa1bd34 Compare January 29, 2021 11:21
@@ -0,0 +1,47 @@
name: Pull request checks
Copy link
Contributor

@xbill82 xbill82 Feb 2, 2021

Choose a reason for hiding this comment

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

is it a good idea to scaffold the github actions? Just wondering, because this is pretty much our internal workflow: is the whole community meant to use it?
At the same time, this is not really harmful, yet very useful to us. I don't really have an opinion, just asking about your strategy here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IMHO Github actions are part of the standard Kuzzle workflow which includes a linter and functional tests (like Vue.js/Webpack does if I remember well)

If they don't want to use it they can just delete the files but IMHO users shouldn't

Copy link
Member

Choose a reason for hiding this comment

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

I partly agree with Luca. Maybe we should however remove the deployment part which can be very different depending on the use case

context: ./backend
file: ./backend/Dockerfile
push: true
tags: registry.gitlab.com/kuzzle/<CHANGE-ME>:latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a place where we give instructions to come here and update these ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, but as we agreed, the deployment part is now removed

@Aschen Aschen merged commit 070d574 into develop Feb 8, 2021
@Aschen Aschen deleted the update-app-template branch February 8, 2021 12:20
@Aschen Aschen mentioned this pull request Feb 8, 2021
Aschen added a commit that referenced this pull request Feb 8, 2021
# [0.18.0](https://github.com/kuzzleio/kourou/releases/tag/0.18.0) (2021-02-08)


#### New features

- [ [#82](#82) ] Add an option to export data in a format compatible with Kuzzle fixtures   ([jenow](https://github.com/jenow))
- [ [#81](#81) ] Add es:migrate command   ([alexandrebouthinon](https://github.com/alexandrebouthinon))

#### Enhancements

- [ [#93](#93) ] Update app template   ([Aschen](https://github.com/Aschen))
- [ [#92](#92) ] Add yaml support for vault   ([Aschen](https://github.com/Aschen))

#### Others

- [ [#77](#77) ] Add Quine example   ([Aschen](https://github.com/Aschen))
---
@Leodau Leodau linked an issue Feb 11, 2021 that may be closed by this pull request
@Aschen Aschen mentioned this pull request Feb 25, 2021
Copy link

🎉 This PR is included in version 1.0.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade the SDK to benefits from improved stacktrace
5 participants