search

winglang / wing

A programming language for the cloud ☁️ A unified programming model, combining infrastructure and runtime code into one language ⚡
https://winglang.io
Other
4.77k stars 189 forks source link

Compiler warnings #1475

Open Chriscbr opened 1 year ago

Chriscbr commented 1 year ago 
          Do we have a mechanism to emit compiler warnings/info?
                // Currently Wing does not support union types (see https://github.com/winglang/wing/issues/977), 
                // so we'll model it the same way as if we saw an "any" in a JSII library.

_Originally posted by @yoav-steinberg in https://github.com/winglang/wing/pull/1397#discussion_r1098328639_

Chriscbr commented 1 year ago

We will probably want warnings in the future, for example to tell users that they have an unused variable

staycoolcall911 commented 1 year ago

Nice! P2 though 😄

github-actions[bot] commented 1 year ago

Hi,

This issue hasn't seen activity in 60 days. Therefore, we are marking this issue as stale for now. It will be closed after 7 days. Feel free to re-open this issue when there's an update or relevant information to be added. Thanks!

github-actions[bot] commented 1 year ago

Hi,

This issue hasn't seen activity in 60 days. Therefore, we are marking this issue as stale for now. It will be closed after 7 days. Feel free to re-open this issue when there's an update or relevant information to be added. Thanks!

MarkMcCulloh commented 8 months ago

From sync discussion on compiler warnings:

Warnings are basically considered "non-fatal errors". By default, warnings cause compilation to fail (non-0 exit code), however, the compiler may still emit (and e.g. run tests) if no errors are present. Even though this fails compilation, The language server should make sure to still treat these as warnings (e.g. yellow text).

A CLI option should be provided to ignore/silence warnings.

github-actions[bot] commented 1 month ago

Hi,

This issue hasn't seen activity in 90 days. Therefore, we are marking this issue as stale for now. It will be closed after 7 days. Feel free to re-open this issue when there's an update or relevant information to be added. Thanks!