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

Remove TOML dependency #82

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Remove TOML dependency #82

merged 2 commits into from
Sep 13, 2023

Conversation

jb3
Copy link
Member

@jb3 jb3 commented Sep 13, 2023

Removes the TOML dependency in favour of using tomllib from the Python 3.11 standard library.

@ChrisLovering
Copy link
Member

@jb3

While we appreciate your contribution, our team has assessed the issue and determined that your PR falls under the category of low priority rather than being critical. This decision is based on our current workload and resource allocation, as we are actively addressing higher-priority tasks and issues.

Rest assured, your PR is in the queue, and we will conduct a thorough review within the next 7-14 working days. We value your commitment to improving our codebase and appreciate your understanding of our prioritization process.

If you have any questions or require further information, please feel free to reach out.

@ChrisLovering ChrisLovering merged commit 4f1bceb into main Sep 13, 2023
4 checks passed
@ChrisLovering ChrisLovering deleted the jb3/remove-toml-dep branch September 13, 2023 09:11
@ChrisLovering
Copy link
Member

ChrisLovering commented Sep 13, 2023

@jb3 I wanted to provide you with an important update regarding your PR.

Upon further review and considering its impact, we have realized that your PR is, indeed, of urgent importance. We have taken immediate action, and I am delighted to inform you that your PR has been successfully merged into our codebase.

We apologize for any previous confusion in our prioritization and thank you for your swift response in addressing this issue. Your contribution has been invaluable to the project.

If you have any questions or need further assistance, please do not hesitate to reach out. Your dedication to our project is greatly appreciated.

@jb3
Copy link
Member Author

jb3 commented Sep 13, 2023

Hi @ChrisLovering,

Wowza! You just made my day brighter than a supernova! 🌟 I'm ecstatic to hear that my PR has been merged and has achieved "urgent importance" status. I guess I should start practicing my superhero theme song now!

No need to sweat the small stuff – I know prioritizing tasks can be as tricky as navigating a corn maze blindfolded. Your quick action and cosmic wisdom in recognizing the PR's importance is much appreciated.

If there's anything else that needs my super-contributor powers or if you just want to discuss the best superhero capes, feel free to shout out. I'm here to rescue code and bring smiles to commit messages!

Thanks a gazillion for the opportunity, and I'm eagerly looking forward to our next coding adventure!

Best regards,
Joe Banks aka "Merge Man" 🦸‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants