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

Adding instructions to README.md and configuring prophet utils unit test #27

Open
wants to merge 21 commits into
base: graalvm-extensions
Choose a base branch
from

Conversation

Nitsua365
Copy link

No description provided.

@Nitsua365 Nitsua365 requested a review from d-kozak January 26, 2023 01:31
@Nitsua365 Nitsua365 removed the request for review from d-kozak January 27, 2023 01:33
@Nitsua365 Nitsua365 requested a review from d-kozak February 1, 2023 02:19
@d-kozak
Copy link

d-kozak commented Feb 8, 2023

Please target https://github.com/cloudhubs/prophet-utils/tree/graalvm-extensions branch instead of master. We will keep the modified version there for now. By targeting master, this PR shows a much bigger diff in terms of changed files than it should.

@d-kozak
Copy link

d-kozak commented Feb 8, 2023

Also please make sure there are no unnecessary merge commits in the history. Which branch did you start from? Seems to be a master branch? Maybe it will be faster to make a new branch from /graalvm-extensions and then cherry pick your changes there, so that the commit history is not polluted. If you are not sure how to do it, we can discuss that during our meeting tomorrow.

@Nitsua365 Nitsua365 changed the base branch from master to graalvm-extensions February 9, 2023 18:22
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.

3 participants