Review of "GNOME Vagrant Indicator" version 16

Details Page Preview

Easily manage your vagrant machines from status area

Extension Homepage
https://github.com/fffilo/gnome-vagrant-indicator

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version Status
17 Active
16 Active
15 Active
14 Active
13 Active
12 Active
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Active
3 Active
2 Active
1 Active

Previous Reviews on this Version

JustPerfection active
This is only supporting 42, so you don't need to use line 405 in `libs/extension/indicator.js`.
JustPerfection posted a review
btw, you are creating objects in global scope which is the same as init (lines 59, 80 and 93 libs/extension/vagrant.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-init-for-initialization I approved this since that's enum like instance but please fix it for the next version.