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

README and minor code fix #598

Merged
merged 2 commits into from Jan 5, 2021
Merged

README and minor code fix #598

merged 2 commits into from Jan 5, 2021

Conversation

Marcono1234
Copy link
Contributor

@Marcono1234 Marcono1234 commented Jan 4, 2021

  • Improves the README by using HTTPS links and fixing some outdated links
  • Fixes unintended String concatenation

Please let me know if the commits should have different messages.


Sorry, but I won't sign a CLA for this. Feel free to close this pull request if you require a CLA for these changes.

@jbonofre jbonofre self-requested a review January 5, 2021 12:58
@jbonofre
Copy link
Member

jbonofre commented Jan 5, 2021

Can you please squash the commits in one ? Thanks !

@jbonofre jbonofre merged commit 9d220c2 into apache:master Jan 5, 2021
@Marcono1234 Marcono1234 deleted the marcono1234/minor-fixes branch January 5, 2021 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants