Skip to content

(Breaking) Get String List#12

Open
ineanto wants to merge 20 commits into
jsixface:masterfrom
ineanto:master
Open

(Breaking) Get String List#12
ineanto wants to merge 20 commits into
jsixface:masterfrom
ineanto:master

Conversation

@ineanto

@ineanto ineanto commented Jul 25, 2023

Copy link
Copy Markdown

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
Copy Markdown
Owner

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

@jsixface

Copy link
Copy Markdown
Owner

Can you update your branch with the latest from master

@ineanto

ineanto commented Mar 18, 2024

Copy link
Copy Markdown
Author

Can you update your branch with the latest from master

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

@ineanto

ineanto commented Apr 15, 2024

Copy link
Copy Markdown
Author

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