Immo Landwerth
Immo Landwerth
This adds two additional rules for dealing with attributes: ### Removing superfluous parentheses It's conventional to omit parentheses when the attribute doesn't take any arguments, for example: ``` CSharp [target:...
We currently analyze the language directly. Maybe we should switch to `IOperation` instead. This would allow us to handle C# and VB from the same code base.
We should consider offering fixers, such as: 1. Suppress and perform runtime check 2. Cross-target 3. Ignore warnings for affected platforms ### Suppress and perform runtime check ```C# int x...
This would allow the developer to understand the reason & condition. Needless to say, this needs to work cases where the exception is constructed from string resources.
See [this policy rule](https://github.com/dotnet/org-policy/blob/master/doc/PR15.md) for more details.
This query return all repos in the org (we have 349 repos): ```C# var query = new Query() .Organization(orgName) .Repositories() .AllPages() .Select(r => new CachedRepo() { Name = r.Name, LastPush...
I have [the following query](https://github.com/dotnet/org-policy/blob/master/src/Microsoft.DotnetOrg.GitHubCaching/CacheLoader.cs#L395-L420): ```C# var query = new Query() .Repository(Var("repoName"), orgName) .Collaborators(first: 100, after: Var("after"), affiliation: CollaboratorAffiliation.Direct) .Select(connection => new { connection.PageInfo.EndCursor, connection.PageInfo.HasNextPage, connection.TotalCount, Items = connection.Edges.Select(e =>...
***Update**: The issue seems to be independent of the query and occurs when the API rate limit is exceeded.* When executing [this query](https://github.com/dotnet/org-policy/blob/master/src/Microsoft.DotnetOrg.GitHubCaching/CacheLoader.cs#L229-L265): ```C# var query = new Query() .Organization(orgName)...
#### Repro This query crashes. The involved repo is public, so you should be able to repro it from your end. Code ```xml ``` ```C# class Program { static async...
A default instance of `IssueFilter` has all properties set to `null`, which in the resulting graph query will compare all the fields to `null`. However, this means it looks for...