Skip to content

#284 Optimization of VC UTBot Quickstart wizard #302

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

Merged
merged 2 commits into from
Jun 30, 2022
Merged

Conversation

alexey-utkin
Copy link
Collaborator

  • support CMake-generated version
  • disabled and checked attribute support for Unix/MS HTML render
  • added client/server version verification at Handshake call
  • deep refactoring .css files
  • remove Test server button and make the mandatory job in background
  • remove useless class WizardHTMLBuilder
  • fix CMake parameter editor

- support CMake-generated version
- `disabled` and `checked` attribute support for Unix/MS HTML render
- added client/server version verification at `Handshake` call
- deep refactoring `.css` files
- remove `Test server` button and make the mandatory job in background
- remove useless class `WizardHTMLBuilder`
- fix CMake parameter editor
@alexey-utkin alexey-utkin requested a review from ladisgin June 29, 2022 14:34
<b>📶 Connection</b>
<span>
<span id="connection_loader">⏳ Connecting...</span>
<span id="connection_success">✔️Successfully pinged server! </span>
Copy link
Member

Choose a reason for hiding this comment

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

add space after emoji

Comment on lines 33 to 36
// vscode.postMessage({
// command: 'dbg_message',
// message: message
// });
Copy link
Member

Choose a reason for hiding this comment

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

uncomment

* Communication with VSCode via API
*/
const vscode = acquireVsCodeApi();
function setupLocalHost() {
Copy link
Member

Choose a reason for hiding this comment

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

choose better name

return host === '127.0.0.1' || host === 'localhost';
}

public static isRemoteScenario(): boolean {
Copy link
Member

Choose a reason for hiding this comment

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

change function appropriately

- fix `isRemoteScenario` for local-remote scenario (remote root != project path)
@alexey-utkin alexey-utkin merged commit 920a8f6 into main Jun 30, 2022
@alexey-utkin alexey-utkin deleted the uta/wizard branch June 30, 2022 07:48
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