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

Updates to gh-actions and tests #13

Merged
merged 29 commits into from
Sep 6, 2023
Merged

Updates to gh-actions and tests #13

merged 29 commits into from
Sep 6, 2023

Conversation

jiajic
Copy link
Member

@jiajic jiajic commented Sep 6, 2023

Separate into full production and simplified development workflows.
get codecov to work
tests now link to GiottoClass version of objects in GiottoData@suite_modular

Separate into full production and simplified development workflows.

Streamline workflow and fix github permissions

production workflow to be updated after dev works
ensure suite_modular branch of GiottoData is used
try not installing GiottoData in actions, relying on the testthat.R script
Edit show functions for packed objects.
Update workflow to specify GiottoClass for python environment installation
- Export `compatible_spatial_network()`
- Export `get_distance()`
- Edit `combine_metadata()` to use merge instead of cbind
- Update `test_gobject.R` to not use conda
- Remove python env build on dev workflow.
- try to silence deprecate_soft() warnings for internal function usage
- separate python env build from GiottoData installation in prod workflow
- try to silence lifecycle for internal function
internals are deprecated for external use, but are still in use
- split off codecov as separate job again due to issues with finding the correct repo location
- try to add lintr
- styler run on classes.R
- fix DT vars in combine_metadata.R
Update checks and gh-actions
@jiajic jiajic merged commit 9e8ce17 into main Sep 6, 2023
2 of 3 checks passed
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.

1 participant