-
Notifications
You must be signed in to change notification settings - Fork 16
Refactor/auto compile #28
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
Refactor/auto compile #28
Conversation
review-deriv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description is too short. You can use this format:
# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.
# Cards
* https://clickup.com/12345
* https://clickup.com/12346
review-deriv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description is too short. You can use this format:
# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.
# Cards
* https://clickup.com/12345
* https://clickup.com/12346
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think rebuild deriv/perl , debian-pg, based on the new perl is not incluced in this task
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed them from card description. we can do that in other cards.
README.md
Outdated
| make | ||
| make install | ||
| ```bash | ||
| VERSION=5.26.1 ./rebuild.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we reference 5.26 here?
b55ab7a to
7c2f241
Compare
Co-authored-by: kathleen-deriv <122329285+lim-deriv@users.noreply.github.com>
7c2f241 to
7c554df
Compare
Compile perl with CI system