Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Align lock file for .zip Installation Instructions, general cleanup #172

Merged
merged 4 commits into from
Mar 23, 2024

Conversation

robertdevine
Copy link
Collaborator

Removal of the {pilot1wrappers} package. Update renv/activate file extension to regulatory technical specification requirement. General tidying.

…val of the {pilot1wrappers} package. Update renv/activate file extension to regulatory technical specification requirement. General tidying.
@robertdevine robertdevine added the fda pilots Link Summary of Pilots and Pilot4 blog post label Mar 23, 2024
@robertdevine robertdevine requested review from bms63 and laxamanaj March 23, 2024 00:20
@robertdevine robertdevine self-assigned this Mar 23, 2024
@robertdevine robertdevine linked an issue Mar 23, 2024 that may be closed by this pull request
@robertdevine
Copy link
Collaborator Author

@laxamanaj , @bms63 - should be all set with the new lock file - had to manually style it since styler does not support. Cleaned all the CRLF stylings by hand - etc.. Good to go. [So this has {pilot3utils} latest v0.0.2 and {pilot1wrappers} removed.] Thx.

Copy link
Collaborator

@bms63 bms63 left a comment

Choose a reason for hiding this comment

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

Thanks for cleaning this up!! We should monitor the website action as it might fail. The action uses the renv.lock to build everything - but we should probably rethink that action anyways.

@bms63 bms63 merged commit a14c076 into main Mar 23, 2024
5 checks passed
@bms63 bms63 deleted the 133-review-lock-file-and-remove-unused-packages branch March 23, 2024 17:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
fda pilots Link Summary of Pilots and Pilot4 blog post
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review Lock file and remove unused packages
2 participants