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

Include project status and related work in readme #700

Merged
merged 5 commits into from
Mar 14, 2024

Conversation

lognaturel
Copy link
Contributor

@lognaturel lognaturel commented Mar 11, 2024

Addresses discussion in #699 and #698

Why is this the best possible solution? Were any other approaches considered?

It makes minimal updates but hopefully helps clarify who is doing what here and why.

What are the regression risks?

None, docs only.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No,

Before submitting this PR, please make sure you have:

  • included test cases for core behavior and edge cases in tests
  • run python -m unittest and verified all tests pass
  • run ruff format pyxform tests and ruff check pyxform tests to lint code
  • verified that any code or assets from external sources are properly credited in comments

README.rst Outdated
Related projects
================

* `xls2xform <https://github.com/mvpdev/xls2xform/>`_: ``pyxform``'s precursor which was rewritten into the library we now know
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@spwoodcock we could add your Dockerfile repo here

README.rst Outdated Show resolved Hide resolved
@lindsay-stevens
Copy link
Contributor

Reworded some of the new content to be a little more assertive, and added a bunch of related projects that I've starred over the years

@lognaturel lognaturel merged commit f524ae7 into XLSForm:master Mar 14, 2024
10 checks passed
@lognaturel
Copy link
Contributor Author

Thank you!

@lognaturel lognaturel deleted the readme branch March 14, 2024 23:23
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.

2 participants