-
Notifications
You must be signed in to change notification settings - Fork 36
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
support virtual variable value #228
Open
iamxiaojianzheng
wants to merge
77
commits into
mistweaverco:feat/variable-values-hover
Choose a base branch
from
iamxiaojianzheng:feat/variable-values-hover
base: feat/variable-values-hover
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
support virtual variable value #228
iamxiaojianzheng
wants to merge
77
commits into
mistweaverco:feat/variable-values-hover
from
iamxiaojianzheng:feat/variable-values-hover
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* chore(docs): add best-practices.md in getting-started * docs: rename --------- Co-authored-by: Marco Kellershoff <[email protected]>
iamxiaojianzheng
requested review from
gorillamoe,
Grueslayer and
treywood
as code owners
September 10, 2024 05:59
Bumps [body-parser](https://github.com/expressjs/body-parser) and [express](https://github.com/expressjs/express). These dependencies needed to be updated together. Updates `body-parser` from 1.20.2 to 1.20.3 - [Release notes](https://github.com/expressjs/body-parser/releases) - [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md) - [Commits](expressjs/body-parser@1.20.2...1.20.3) Updates `express` from 4.19.2 to 4.20.0 - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.19.2...4.20.0) --- updated-dependencies: - dependency-name: body-parser dependency-type: indirect - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat(ci): add docs linter * chore(docs): fix vale linter errors
* fix(shlex): make split function more robust And more close to Python original. Now it supports like `curl https://example.com` (without quotes), or ` curl 'https://example.com'` (with a space at the start). 1. Add a test data from CPython test_shlex.py 2. Reimplement Python code generating a test cases. 3. Mark a few tests as broken and move to separate test case - probably not worth fixing. 4. Add a fix for curl case - set whitespace_fix to true, this is what Python shlex.split does. 5. Avoid `goto continue` by introducing a continue boolean flag in a parsing code. Resulting code is ugly, but it works. * fix(lint): 💄 * use SHLEX.split method in broken test case too --------- Co-authored-by: Marco Kellershoff <[email protected]>
gorillamoe
requested changes
Sep 12, 2024
…co#239) * Output path to curl executable and version in checkhealth * Linter suggestions (spaces / quotes) --------- Co-authored-by: Jan Stocker <[email protected]>
Form-data is now handled correctly. Also added the possiblity to add and send binary files in form-data. Added body_display which is basically a body that is meant to be displayed in some way (for now it's only stripped from binary data).
remove nofile, otherwise ts folds do not work in lazy.
So it has not to be created on the fly and is not causing issues with R/O FS. X-Ref: mistweaverco#282
* fix(tmp): use `stdpath('cache')` for tmp files This should fix: mistweaverco#282 * Add windows testrunner dockerfile * fix(ci): workflow file * refactor(ci): add optional windows test pipeline * feat(ci): windows * fix(fs): join_paths on windows * fix(ci): windows cache * fix(fs): gsub returns two params * debug(ci): windows caching * feat(tests): windows fs.join_paths * refactor(ci): tests
* Keep alternative-file * Fix spacing --------- Co-authored-by: Jan Stocker <[email protected]>
iamxiaojianzheng
force-pushed
the
feat/variable-values-hover
branch
from
October 24, 2024 03:38
b2f305d
to
25be105
Compare
This reverts these quickfix commits: mistweaverco#279 mistweaverco#286
) Co-authored-by: anonymous <[email protected]>
This is a workaround for a bug/weird behaviour in neovim: neovim/neovim#31107 Once this has been merged: neovim/neovim#31109 We can revert this workaround. Fixes mistweaverco#304
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.