-
-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature Requests #34
Comments
Request #1: Cover inline table valued functions Statements inside inline table valued functions are not currently covered, the sp_statement_completed event doesn't include inline tvf's but we do get a intent-shared transaction lock so we could include those in the trace and show whether an inline tvf has at least been called. They can't contain branches in so they would either be 0% or 100% covered. Possible issues to consider: Do different isolation modes change the lock types? This was copied from: #30 |
Request #2: Corbertura output files support done - included in 0.5.0: |
Request #3: html output: navigation buttons to go to top of page done - included in 0.5.0: |
Request #4: Sql 2019 |
Request: #5: Sql on linux support (might just be testing) |
Request #6: Configure timeout done - in 0.5.0: |
Do we have a resolution for Request #5 please? |
@Athi2019 no sorry, I haven't tried covering t-sql on a sql on linux instance - if you have and it doesn't work it would be great if you can add any details of exceptions etc. |
@GoEddie I am trying to cover t-sql on a windows sql instance . Please refer this link for the details: |
Hi @Athi2019 sorry I misunderstood, I see now :) The issue you have is a few days work and I don’t have the capacity at the moment for that amount of OSS work. If it is important to you and you have a budget I would be happy to discuss sponsorship to get it working for you, if you would like to pursue this please reach out to me ed@agilesql.co.uk Ed |
Please can you add an azure-pipeline examples to this repo.. I am using with Azure and I am not sure I am doing it right because report generator shows files missing when I drill down. |
Request: Output report files in UTF-8 by default. I had issues uploading the OpenCover XML to sites like https://codecov.io since it wasn't in UTF-8. I'm now manually converting before uploading, but it would be nice if that was default or an option. A low lift would be to include this in the example |
@persistence911 there is now a github action that works, also see my open sourced project https://dev.azure.com/funktechno/_git/dotnet%20ci%20pipelines?path=%2Fazure-pipelines.yml for running a sql server instance in an azure devops pipeline |
If you have any feature requests, add a comment here with as much detail - when I work on the next version then I will consider them.
If you want to see a feature implemented it, add a comment with the feature number and a +1
The text was updated successfully, but these errors were encountered: