DotNetAnalyzers / AspNetCoreAnalyzers

MIT License
62 stars 4 forks source link

Require type constraints? #63

Open JohanLarsson opened 5 years ago

JohanLarsson commented 5 years ago

Before:

namespace AspBox
{
    using Microsoft.AspNetCore.Mvc;

    [Route("api/values")]
    [ApiController]
    public class ValuesController : ControllerBase
    {
        [HttpGet("{id}")]
        public IActionResult Get(int id)
        {
            ...
        }
    }
}

After:

namespace AspBox
{
    using Microsoft.AspNetCore.Mvc;

    [Route("api/values")]
    [ApiController]
    public class ValuesController : ControllerBase
    {
        [HttpGet("{id:int}")]
        public IActionResult Get(int id)
        {
            ...
        }
    }
}
lloydjatkinson commented 5 years ago

This is a good idea