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

Bootleg ci wip #8511

Closed
wants to merge 23 commits into from
Closed

Bootleg ci wip #8511

wants to merge 23 commits into from

Conversation

ganthern
Copy link
Contributor

No description provided.

abdulkadir-tutao and others added 22 commits February 6, 2025 13:28
the progress estimation for failed imports was not stopped.
this gave the impression that the import was still ongoing.

we now reset the state if an error happens during import preparation
and will pause the running import if an async error happens.
this will allow the user to try to resume it and see the snackbar or
notification if the error happens again.

also adds some local state cleanup if the import fails before being
set up on the server
previously, we didn't increment the counter for all failure cases,
leading to
the finished import showing 0 failed mails even though some actually
failed.

we still leave some edge cases where we fail to import a mail but also
fail to mark it as failed.
- do not depend on un-needed tokio features
- move rand to Cargo.toml
- use latest uniffi-bindgen from crates.io
commands like `cargo test`, `cargo clippy` or `cargo check` now require
the `--all` or `--package` flags to choose which workspace members they
should run for.
this prevents the mobile apps from building node-mimimi
mlock is used by sqlcipher but is not allowed on our test containers
see sqlcipher/sqlcipher-android#48 (comment)
see systemd/systemd#9414 (comment)
this will improve incremental compilation times since
we don't use multiple target folders anymore.

Co-authored-by: sug <[email protected]>
apple mail exports directories that masquerade as mbox files, but
at least the structure is predictable.
we can cut the time for the checks by 75%-80%
by doing this.

* exclude generated files from swiftlint/formatting
* add a pre-push hook to generate a link for the ci check & merge
* set the build result to unstable if run with DRY_RUN=true
* remove Rustflags, they produce unneeded rebuilds
* remove x86_64 target to speed up build
* improve pre-push hook
* run gradlew test for arm64 only in ci
* also check the tuta-sdk folder for FIXMEs
* don't run ios tests on sdk changes
* make ios builds mutually exclusive on the same machine
  xcode likes to use a global directory for DerivedData and really
  doesn't like if the files there change while a build is running.
  to make sure our test workflow and the ios app builds don't clash, we
  define lockable resources per mac machine.
* add rust style and lint to merge.groovy
* make it possible to prune the ci workflow

Co-authored-by: sug <[email protected]>
Co-authored-by: Kinan <[email protected]>
Co-authored-by: map <[email protected]>
Co-authored-by: nig <[email protected]>
@ganthern ganthern force-pushed the bootleg-ci branch 2 times, most recently from 8fb1b7b to a60cbbb Compare February 10, 2025 10:53
this commit can be reverted if we decide we don't need github workflows
anymore.
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.

4 participants