Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove unnecessary activation events #356

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

hwayne
Copy link
Contributor

@hwayne hwayne commented Dec 20, 2024

Whenever I open the extension, vscode complains that commands don't need to be added to activationEvents.

image

This PR fixes that terrible oversight. The extension works normally without them.

Whenever I open the extension, vscode complains that commands don't need to be added to activationEvents. This commit fixes that terrible oversight.

Signed-off-by: Hillel <h@hillelwayne.com>
@FedericoPonzi FedericoPonzi merged commit 3731be5 into tlaplus:master Dec 20, 2024
4 checks passed
@lemmy
Copy link
Member

lemmy commented Dec 21, 2024

For the record, there was a bug where the TLA+ commands of the extension wouldn’t appear unless some .tla file was opened in the editor. Consequently, it became impossible to use the REPL alone. However, I don’t seem to be able to reproduce this issue anymore. Therefore, I’m leaving it here just because this issue is about activation and in case the problem resurfaces.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants