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

Create workflow for build and release #326

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShivamMenda
Copy link
Contributor

Description

Created a CI Pipeline for build and release with Semantic versioning depending on the commit message

Fixes #319

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

This is yet to be tested because a PAT needs to be added to the project that can perform this workflow and then to be checked whether the release has been created properly with the workflow

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Maintainer Checklist

@ShivamMenda
Copy link
Contributor Author

@chandansgowda @Aarush-Acharya I've been trying to run this workflow but the app does not build correctly.
Screenshot 2024-05-25 at 1 21 17 AM

@Aarush-Acharya
Copy link
Contributor

please do check if the current state of the app works on your local

@ShivamMenda
Copy link
Contributor Author

Sure @Aarush-Acharya Will check

@ShivamMenda
Copy link
Contributor Author

@Aarush-Acharya I think it's an issue with the code written for the health check, if you see in the logs, The health check part is where the build is breaking.

@Aarush-Acharya
Copy link
Contributor

Aarush-Acharya commented May 25, 2024

yes @ShivamMenda it is from a PR recently merged #213 for checking if the appwrite server is down if this is not working on CI CD flow i feel it would not work on local as well, just try building the current state of the app on your local, and see if it throws this error

If it does
please try to resolve it

else if it works
check your dart version and flutter version and check the dart and flutter version in environment the build release is taking place

@ShivamMenda
Copy link
Contributor Author

@Aarush-Acharya Will look into it, if it fails, I will open a PR to fix the Server check issue and then you can merge the CI CD PR.

@Aarush-Acharya
Copy link
Contributor

Aarush-Acharya commented May 25, 2024

@ShivamMenda understood for reference and more context view #213

@Aarush-Acharya
Copy link
Contributor

Aarush-Acharya commented May 25, 2024

Hey @ShivamMenda let me know if any updates on solving the current error in the repo, please do have that one done ASAP as it will affect other contributors

@Aarush-Acharya
Copy link
Contributor

Hey @ShivamMenda, the current error was solved we reverted changes, please do complete this asap and let me know

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.

Setup CI/CD for building the app and creating a release.
2 participants