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

chore(java): move the source code to the correct folder #429

Merged
merged 6 commits into from
Apr 28, 2022

Conversation

millotp
Copy link
Collaborator

@millotp millotp commented Apr 26, 2022

🧭 What and Why

To respect gradle convention and make room for some future test, we move the source code to:
algoliasearch-core/src/main/java/com/algolia (not at all convoluted)

🧪 Test

CI

@millotp millotp self-assigned this Apr 26, 2022
@netlify
Copy link

netlify bot commented Apr 26, 2022

Deploy Preview for api-clients-automation canceled.

Name Link
🔨 Latest commit 851167a
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/626a4f7bb9ed040008d9c97d

@algolia-bot
Copy link
Collaborator

algolia-bot commented Apr 26, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.

@shortcuts
Copy link
Member

shortcuts commented Apr 26, 2022

I really recommend this for those large actions: https://gist.github.com/emiller/6769886 so you don't lose anything (moving file does not exists in git, you lose history as if it was deleted + added) but still manage to apply the changes.

Otherwise, we can rely on the CI

@millotp
Copy link
Collaborator Author

millotp commented Apr 26, 2022

Not usable without using push force on main, it's not usable for our setup

@shortcuts
Copy link
Member

shortcuts commented Apr 26, 2022

I think it works on a branch because you'll merge the history on main but I'm not sure.

But since it's the same, we can let the CI do the job. (It will allow allow us to debug if deleted files are correctly pushed)

@millotp millotp requested review from a team, eunjae-lee and shortcuts and removed request for a team April 27, 2022 13:28
Copy link
Contributor

@eunjae-lee eunjae-lee left a comment

Choose a reason for hiding this comment

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

Let's go!

@millotp millotp enabled auto-merge (squash) April 28, 2022 08:33
@millotp millotp merged commit b4ff3ff into main Apr 28, 2022
@millotp millotp deleted the chore/move-java-again branch April 28, 2022 08:36
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