-
Notifications
You must be signed in to change notification settings - Fork 4
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
Deleted X-UA-Compatible meta tag #21
Open
ghost
wants to merge
30
commits into
onivim:master
Choose a base branch
from
unknown repository
base: master
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
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
* Remove vscode-ripgrep to avoid rate limits * Get tests green; get CI set up * Update yarn.lock file * Update build script * Remove unit tests for now
* Start a test case to validate extHost sends initialize via json-rpc * Almost building... * Get very basic initialize test passing * Update test description * Get initialiation path green * Close server when done * Remove node-pty dependency * Tweaks
* Add sandbox for testing out the extension host API * Start testing out basic command API * Get message from extension * Remove superfluous logging * Remove additional logging * Move test.js -> test.ts * Factor out extension host helper * Don't run extension host helper as a test * Factor tests to use new ExtensionHost model * Add test case validating plugin activation * Get activation test green
* Clean up tests * Add test to validate command execution
* Add test for opening / closing document * Fix slash separator on non-windows platforms
* Add some infra to create a package build * Set up to create npm package * Bump version -> 1.33.2 * Successful package publish! * Add build step for packaging
* Remove SQLite dependency * Fix packages
* Add LSP extension * Move oni test extensions to separate folder * Get some initial diagnostics! * Remove outputAppender for spdlog * Fix watch command * Get tests green again * Patch console.log/warn/error so they don't blow up our stdio RPC protocol * Refactor API for documents in tests * Switch one test to use new helpers * Add test for diagnostics * Update build * Fix value
* Add initial activation-event test extension * Add minimal activation * Add test case for activation events
* Start test for completions * Formatting * Add support for responses, get suggestions * Format * Remove yarn install * bump
* Exercise failure case with textDocumentPosition, and fix in test * Disable proposed extensions
…18) * Add test case for getting workspace.rootPath * Fix test for Windows * Test changing workspace * Get tests green
…ing config (#19) * Add test case for getting workspace.rootPath * Test case for configuration * Fix merge in extension.js * Hook up configuration settings / changes
* Bring back node-pty, update lockfile * Revert changes to terminalProcess.ts * Update node to pick up latest 10.x * Experiment with npmrc * Revert "Experiment with npmrc" This reverts commit a0db22b. * Remove yarnrc * Remove electron checks * Remove gc-signals * it...built? * Logging / hacks to get terminal service available * Revert changes to mainThreadHeapService * Add test case for terminal execution * Simplify first test to just check for process exit * Fix bug with windows terminal output * Add test for / * Clean up, add config neeeded to enable terminal * Get tests green with configuration update * Remove logging * Save sandbox test
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.
There is no need: https://stackoverflow.com/questions/26346917/why-use-x-ua-compatible-ie-edge-anymore