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

Styling changes for Gradio Workbook #827

Closed
wants to merge 9 commits into from
Closed

Styling changes for Gradio Workbook #827

wants to merge 9 commits into from

Conversation

Comment on lines +5 to +6
name = "python-aiconfig-test"
version = "1.1.25"
Copy link
Contributor

Choose a reason for hiding this comment

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

For future, we don't need to keep changing this. We can have two separate repos and simply when we do pip install we explicitly pull from our test repo:

pip3 install --index-url https://test.pypi.org/project/python-aiconfig-test/

^We can use python-aiconfig instead of python-aiconfig-test

I supposed if we are going to update the the version number to match pytest anyways, it's not that much different to change package name so that's fine too
cc @jonathanlastmileai


# If you want to upload to prod , run pypipublish.sh.

python3 -m twine upload --repository testpypi dist/*
Copy link
Contributor

Choose a reason for hiding this comment

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

cc @jonathanlastmileai can you change your PR from "draft" in #807 so we can land and keep changes separate here?

@zakariassoul
Copy link
Contributor

@saqadri can you post a video or screenshots of the editor?

Successfully loads the local editor as a VSCode webview
Now that our actual local-editor works, no need to use the sample webview-ui to test the extension
Get the aiconfig from the VSCode textdocument:

* The extension sends an `update` request to the webview when the TextDocument changes.

* The webview handles that by updating the state of the AIConfig, which will trigger a re-render.
Delete an older experiment of a vscode extension.

Main difference about this is it uses webpack for the build step but that's most likely unnecessary for our case.
Update content security policy to allow external images to render
Plus publish script for test-pypi
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.

3 participants