Skip to content

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.

4 participants