Skip to content

First pass at Python implementation#25

Closed
adriangb wants to merge 6 commits into1Password:mainfrom
adriangb:python
Closed

First pass at Python implementation#25
adriangb wants to merge 6 commits into1Password:mainfrom
adriangb:python

Conversation

@adriangb
Copy link
Contributor

@adriangb adriangb commented Nov 27, 2022

Todo:

  • A lot of cleanup, this is pretty hacky
  • Fix line spacing
  • Make ordering of globals deterministic (the generated globals are not consistently ordered, which obviously makes tests fail) Done
  • Consider vendoring / implementing a topological sort

@snowsignal
Copy link
Contributor

Thanks for your work on this. Since this is still a draft PR, and it's been stale for a while, I'm closing it. If you want to resume work on this, feel free to open this PR again. FYI, we'll probably incorporate your changes into an eventual complete Python implementation. Again, thanks for the work you've done 😄

@snowsignal snowsignal closed this Oct 24, 2023
@adriangb
Copy link
Contributor Author

No worries. I'm terribly sorry to have started this and then let it languish.

hculea added a commit to hculea/typeshare that referenced this pull request Apr 30, 2024
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