Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Fix module import error by using typing.List #33

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

Cassius0924
Copy link

WHAT

CopilotC-Nvim#18
neovim/pynvim#560

WHY

Using list in Python 3.8 results in an error : TypeError: 'type' object is not subscriptable

HOW

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • Linter
  • Tests
  • Review comments
  • Security

Copy link

changeset-bot bot commented Jan 30, 2024

⚠️ No Changeset found

Latest commit: 24789ee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@jellydn
Copy link
Owner

jellydn commented Jan 30, 2024

Thanks @Cassius0924 LGTM!

@jellydn jellydn merged commit 920380e into jellydn:main Jan 30, 2024
4 checks passed
@jellydn
Copy link
Owner

jellydn commented Jan 30, 2024

@all-contributors add @Cassius0924 for code

Copy link

@jellydn

I've put up a pull request to add @Cassius0924! 🎉

jellydn pushed a commit that referenced this pull request Jan 30, 2024
jellydn added a commit that referenced this pull request Feb 2, 2024
* Intergrate ziontee113/CopilotAgent.nvim (#27)

* feat: add libraries from ziontee113/CopilotAgent

* feat: add chat handlers from CopilotAgent

* feat: add "mycopilot" module from CopilotAgent

* feat: add "tools" module from CopilotAgent

* feat: add copilot-agent.py

* feat: conditionally check for tiktoken module

* Add renamed CopilotAgent commands to README.md (#29)

* ref: rename commands to prefixed with CopilotChat

* docs: add renamed CopilotAgent commmands

* docs: add default key mapping

---------

Co-authored-by: Dung Duc Huynh (Kaka) <[email protected]>

* docs: add note about the canary branch

docs: fix keymap for in-place command

* chore: update dict

* docs: add demo videos for Fold & Inplace (#31)

* chore: remove testing commands and layout component

fix: remove matrix testing code

* docs: remove copilot chat vspilt command

* feat: add toggle layout and new split preset commands

* revert: add internal comands back for inplace chat

* chore: add a note about chat handler

* feat: show spinner on processing

* feat: show key binding on help section

* docs: fix usage for InPlace command

* docs: add demo for token count and demo

* feat: add C-h to toggle help section

* docs: add usage for InPlace command

* chore: sync fork

* refactor: create dedicated function for help text with layout

* feat: add option for hide or show the help text on InPlace

* Fix module import error by using typing.List (#33)

Co-authored-by: Cassius0924 <[email protected]>

* docs: add Cassius0924 as a contributor for code (#34)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>

* feat: add user prompt and clear history on InPlace command

* docs: fix usage for install manually

* feat: add new keymap for change system prompt

* feat: add debug flag

* docs: add debug file path when enable debug mode

* docs: notify end user to run UpdateRemotePlugins command after install plugin

* chore: sync fork

Add system prompt to ask
Handle bad error code after calling post

* chore: sync fork and remove unused

* docs: add Discord link

* chore: sync fork

* refactor!: drop CC commands

* docs: remove branch on usage

---------

Co-authored-by: Trí Thiện Nguyễn <[email protected]>
Co-authored-by: He Zhizhou <[email protected]>
Co-authored-by: Cassius0924 <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants