stan-dev / cmdstan

CmdStan, the command line interface to Stan
https://mc-stan.org/users/interfaces/cmdstan
BSD 3-Clause "New" or "Revised" License
210 stars 93 forks source link

Resolve some unused variable warnings #1199

Closed WardBrian closed 1 year ago

WardBrian commented 1 year ago

Submisison Checklist

Summary:

Resolves some unused variable warnings shown in make build

Copyright and Licensing

Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):

By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

WardBrian commented 1 year ago

@serban-nicusor-toptal This is getting an error from the quality gate I don't understand, could you take a look?

serban-nicusor-toptal commented 1 year ago

Hmh, weird, it's complaining about some regex I think:

java.util.regex.PatternSyntaxException: Dangling meta character '*' near index 0

Checking ...

serban-nicusor-toptal commented 1 year ago

My first guess is that something changed in the output and the parser of the warnings plugin does not like it. Looks like this line is failing the pipeline excludeMessage("*'sprintf' is deprecated*") https://github.com/stan-dev/cmdstan/blob/develop/Jenkinsfile#L234C37-L234C76 I hope this is not because of a plugin update as that will affect us in all pipelines. ( Looks like it's not )

serban-nicusor-toptal commented 1 year ago

I think if we change that to: .*'sprintf' is deprecated.* will make the regex valid. I'm not entirely sure why it failed now tho.

WardBrian commented 1 year ago

Ok, the odd CI failures seem sorted. Mind giving this a review now @mitzimorris?