Skip to content

Conversation

@ineanto
Copy link

@ineanto ineanto commented Jul 25, 2023

Hello. I had initially intended to keep this build to myself but I'm opening this pull request in sight of the requests to introduce a getStringList() method. I'm fully aware that in addition to the new method, I've made a number of (questionable) breaking changes.

These include:

  • Removing every mention of static methods (README for further information)
  • Refactor the documentation
  • Remove the CodeQL analysis GitHub action
  • and more.

If those changes don't follow the vision you have for this project, I'm fully ok with you rejecting this pull request.
Thank you! (Fixes #11)

@jsixface
Copy link
Owner

Hey, thanks for the PR. I will review it and respond later this week.

@jsixface
Copy link
Owner

Can you update your branch with the latest from master

@ineanto
Copy link
Author

ineanto commented Mar 18, 2024

Can you update your branch with the latest from master

Hello! I'm quite busy this week but I'll keep you posted

@ineanto
Copy link
Author

ineanto commented Apr 15, 2024

Hi. I merged with upstream and bumped to version 1.3.0 inside the README file according to the Gradle build file!

Edit:
I also added back the notice for returning users upgrading :)

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.

Add getStringList method

2 participants