summaryrefslogtreecommitdiffstats
path: root/docs/contributing/editors/vscode.rst
blob: 9830dcbe4f996e364894c84f6e4666a9cf57bf2e (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
Visual Studio Code
==================

.. toctree::
   :maxdepth: 1
   :glob:

General Knowledge
-----------------

`VSCode <https://code.visualstudio.com/>`__ is a multi-platform open-source programming editor developed by Microsoft and volunteers.
It has support for many programming languages using extensions.
This is the recommended editor for Firefox development.

For more general information on the VSCode project see the `repository <https://github.com/Microsoft/vscode/>`__.

Recommended extensions
----------------------

VS Code provides number of extensions for JavaScript, Rust, etc. By default,
Firefox source tree comes with its own set of recommendations of Visual Studio
Code extensions. They will be offered when you first open the project.

If you need to refer to them later, the extensions are listed in
`.vscode/extensions.json <https://searchfox.org/mozilla-central/source/.vscode/extensions.json>`__.

For Rust development, the `rust-analyzer <https://marketplace.visualstudio.com/items?itemName=matklad.rust-analyzer>`__ extension is recommended.
`See the manual <https://rust-analyzer.github.io/manual.html>`__ for more information.

Getting setup
-------------

Close `VS Code` if it is already open, then build the configuration for `VS Code`
by simplying running from the terminal:

.. code::

   ./mach ide vscode

This will automatically set some of the recommended preferences for the workspace,
and if you are set up for a full build, it will enable clangd and rust integrations.

If successful, `VS Code` will open at the end. You do not need to run this command
every time to open `VS Code`, you may open it in the normal way.

If you are running full builds, the command above will set up the `Clangd`
integration so that subsequent invocations of ``./mach build`` run and update the
integration.

.. note::

   If `VS Code` is already open with a previous configuration generated, please make sure to
   restart `VS Code` otherwise the new configuration will not be used, and the `compile_commands.json`
   needed by `clangd` server will not be refreshed. This is a known `bug <https://github.com/clangd/vscode-clangd/issues/42>`__
   in `clangd-vscode` extension

Ignore Files in Mercurial Repositories
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

When using Mercurial in mozilla-central, VS Code will treat your build directories as ordinary directories by default, causing some undesirable behavior including long indexing times, Go to Definition will open files in the build directory instead of the source tree, and Search Files by Name will find duplicate files from the source tree and the build directory (note: when using Git, VS Code will not do this since it reads ``.gitignore``). You can follow these directions to have VS Code largely ignore your build directories:

#. Go to Preferences -> Settings
#. Search "exclude" in the Settings
#. (optional) Select "Workspace" below the search bar to only change this setting for the mozilla-central repository
#. Under "Files: Exclude", click "Add Pattern", type ``obj-*`` (assuming your build directory names start with the default text, ``obj-``), and click "OK"
#. Repeat the step above for the "Files: Watcher Exclude" setting
#. Reload VS Code: the easiest way to do this is to quit and reopen it.

Despite excluding the build directories above, Go to Definition will still correctly open files that only appear in the build directory such as generated source code. See `Bug 1790517 <https://bugzilla.mozilla.org/show_bug.cgi?id=1790517>`_ for our effort to automatically exclude the build directories.

Recommended Preferences
~~~~~~~~~~~~~~~~~~~~~~~

.. note::

   These are automatically set when running ``./mach ide vscode`` but may be
   changed manually. These are set only for particular file types.

* ``"editor.formatOnSave": true``
    * This will turn on automatically fixing formatting issues when you save a file.
* ``"editor.defaultFormatter": "esbenp.prettier-vscode"``
   * This sets the default formatter to prettier using the recommended prettier
     extension.

``*.jsm`` and ``*.sjs`` file extensions should also be associated with JavaScript:

.. code::

   "files.associations": {
      "*.jsm": "javascript",
      "*.sjs": "javascript",
   },

C/C++ Features and Support
--------------------------

For C++ support we offer an out of the box configuration based on
`clangd <https://clangd.llvm.org>`__.

Leveraging the `clang` toolchain compiler we now have support in the IDE for the following features:

**1.** Syntax highlighting

**2.** IntelliSense with comprehensive code completion and suggestion

.. image:: ../img/auto_completion.gif

**3.** Go-to definition and Go-to declaration

.. image:: ../img/goto_definition.gif

**4.** Find all references

.. image:: ../img/find_references.gif

**5.** Open type hierarchy

.. image:: ../img/type_hierarchy.gif

**6.** Rename symbol, all usages of the symbol will be renamed, including declaration, definition and references

.. image:: ../img/rename_symbol.gif

**7.** Code formatting, based on `clang-format` that respects our coding standard using the `.clang-format` and `.clang-format-ignore` files. Format can be performed on an entire file or on a code selection

.. image:: ../img/format_selection.gif

**8.** Inline parsing errors with limited auto-fix hints

.. image:: ../img/diagnostic_error.gif

**9.** Basic static-code analysis using `clang-tidy` and our list of enabled checkers. (This is still in progress not all checkers are supported by `clangd`)

Clangd-specific Commands
------------------------

Clangd supports some commands that are specific to C/C++:

.. code::

    "clangd.switchheadersource"

This command navigates from the currently open header file to its corresponding source file (if there is one), or vice versa.

This command can be invoked from the command menu (activated via ``F1``), or using its keybinding of ``Alt+o`` (``Alt+cmd+o`` on Mac). The keybinding can also be customized in ``Keyboard Shortcuts``.

Remote Development over SSH
---------------------------

VS Code provides an `extension <https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-ssh>`__ that lets you use any remote machine with a SSH server as your development environment. No matter if it's Linux based, macOS or Windows, as long as the target machine offers a SSH connection, it can be used for development.

No source code needs to be on your local machine to use VS Code remotely since the extension runs commands and other extensions directly on the remote machine.

In order to setup a connection please follow these steps:

**1.** Open VS Code and select from the left side panel ``Remote Explorer``

.. image:: ../img/remote_explorer.png

**2.** From the ``Remote Explorer`` panel select ``SSH Targets`` and click on ``Add`` and enter the connection details

.. image:: ../img/remote_explorer_add.png

.. image:: ../img/remote_explorer_add_wind.png

**3.** Click on the connection that you just configured at the previous step

**4.** Finally you should be connected to the desired remote SSH server

.. image:: ../img/connection_done.png

Please note that during the first connection VS Code will install itself remotely and also install all of the needed dependencies.



Filing Bugs
-----------

Bugs should be filed in the `Firefox Build System` product under `Developer Environment Integration`, preferably blocking `Bug 1662709 <https://bugzilla.mozilla.org/show_bug.cgi?id=1662709>`__.