Warnings during build/test
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 45/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Stale
- Domain
- build-system, compilers, testing-qa
Research direction
Start by running the build and tests, then inspect SqlDom/UTSqlScriptDom.csproj for the MSB3884 warning and TSql160ParserInternal_final.cs around line 79827 for the unused-variable warning. Check the non-public TestClass warning for UTSqlScriptDom.DataRetentionSqlScriptDomTests as well. Done means the listed warnings, and any additional warnings found during the run, are resolved without introducing new ones.
Written by the indexing model from the issue text.
Description
There's a number of compiler warnings during build/test. It'd be good to get these fixed now so new warnings are easier to notice.
-
UTA001: TestClass attribute defined on non-public class SqlStudio.Tests. UTSqlScriptDom.DataRetentionSqlScriptDomTests -
MSB3884: Could not find rule set file "AllRules.ruleset". [C:\src\SqlScriptDOM\Test\SqlDom\UTSqlScriptDom.csproj::TargetFramework=net462] -
TSql160ParserInternal_final.cs(79827,8): wa rning CS0219: The variable 'vHasColumnList' is assigned but its value is never used [C:\src\SqlScriptDOM\SqlScriptDom\Microsoft.SqlServer.TransactSql .ScriptDom.csproj::TargetFramework=net6.0]
(there may be others, these were just the initial ones I noticed)
- Dominant language
- GAP
- Stars
- 277
- Forks
- 43
- Avg merge
- 6d 17h
- Merged PRs (30d)
- 3
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/SqlScriptDOM
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
microsoft/SqlScriptDOM#228 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
microsoft/SqlScriptDOM#183 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 56/100
microsoft/SqlScriptDOM#226 · 1 reaction ·
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
microsoft/SqlScriptDOM#225 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 58/100
microsoft/SqlScriptDOM#224 ·
All issues in microsoft/SqlScriptDOM
Similar issues
-
Update to NCCL 2.32 Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
conda-forge/nccl-feedstock#166 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
dsx-ai-factory/infra-controller#6694 ·
-
bug build
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
facebookincubator/velox#19143 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
pydantic/pydantic-ai#8636 ·
-
0.kind: enhancement 9.needs: package (update)
Difficulty 2/5 1-3 hours Newbie friendliness 74/100