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

Patch ci #531

Closed
wants to merge 5 commits into from
Closed

Patch ci #531

wants to merge 5 commits into from

Conversation

munoztd0
Copy link
Collaborator

@munoztd0 munoztd0 commented Jun 6, 2023

the min and max candle limits where inversed and where thus not passing the tests
updated r-lib actions to v2
added David to maintainer
fix type in decription names and roles
@munoztd0 munoztd0 self-assigned this Jun 6, 2023
@munoztd0
Copy link
Collaborator Author

munoztd0 commented Jun 6, 2023

Still need some work to fix the pipeline for pull request.. I'm on it

…rrrors

changed the error check function of rmdcheck to faisl only on errrors

pipelein might breank on warnings and that has to be fixed in the future
@munoztd0 munoztd0 closed this Jun 8, 2023
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.

None yet

1 participant