Added missing copyright header

[insomniacslk]

Signed-off-by: Andrea Barberio insomniac@slackware.it

[insomniacslk]

Hopefully this won't create any problem with goose's parsing of that file

[marcoguerri]

Bah, this again, sorry. Doesn't the linter run as part of the integ test run in the container? Probably we already discussed this, I forgot what the outcome of this discussion was.

Hopefully this won't create any problem with goose's parsing of that file

Tested locally and it doesn't interfere with the parsing, anyway the integ test run will exercise the migration so it should confirm.

[codecov]

Codecov Report

Merging #188 (0141ee0) into master (efb4e54) will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
+ Coverage   62.06%   62.14%   +0.07%     
==========================================
  Files          82       82              
  Lines        3883     3883              
==========================================
+ Hits         2410     2413       +3     
+ Misses       1186     1183       -3     
  Partials      287      287              

Flag	Coverage Δ
integration	59.29% <ø> (+0.18%)	⬆️
integration_storage	100.00% <ø> (?)
unittests	27.61% <ø> (+0.18%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
plugins/teststeps/teststeps.go	78.87% <0.00%> (+4.22%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update efb4e54...0141ee0. Read the comment docs.

[insomniacslk]

Bah, this again, sorry. Doesn't the linter run as part of the integ test run in the container? Probably we already discussed this, I forgot what the outcome of this discussion was.

The public linter ignores non-go files while the internal one doesn't. I can adjust the public linter to include sql, one sec

[insomniacslk]

Updated, but I have to change the regexp too, to match the sql-style comments, one sec

[insomniacslk]

Updated, but I have to change the regexp too, to match the sql-style comments, one sec

Interestingly, it returns success, but it should fail because of the different comment style

[insomniacslk]

not sure why this is happening, but sounds like a bug in checklicenses

[insomniacslk]

will look at that tomorrow

[insomniacslk]

@marcoguerri if that's OK I will merge this to fix the copyright header, and file an issue for checklicenses

[marcoguerri]

Sorry for the latency, yes, that's perfectly fine.