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.

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.