Define marker attributes on a external assembly

[skarllot]

The issue or feature being addressed

• Jab is not working with InternalsVisibleTo #122
• Removing unnecessary code from output assembly #150

Details on the issue fix or feature implementation

• Create the assembly named Jab.Attributes to define marker attributes

[skarllot]

The package content was changed to:

• 📂 analyzers/
  • 📂 dotnet/
    • 📂 roslyn3.11/
      • 📄 Jab.Attributes.dll
      • 📄 Jab.Roslyn3.dll
    • 📂 roslyn4.0/
      • 📄 Jab.Attributes.dll
      • 📄 Jab.dll
• 📂 lib/
  • 📂 netstandard2.0/
    • 📄 Jab.Attributes.dll
• 📄 README.md

[pakrym]

Thank you for your contribution! One of the benefits of Jab, as I see it, is not needing any external dependencies.

There are reasons to have attributes as a separate assembly (like supporting InternalsVisibleTo), but it should be an opt-in rather than the default.

I was thinking about having Jab.Attributes as a separate package and the Attributes.cs injection, conditional on whether the package is already included.

[skarllot]

Done, the Jab no longer depends on Jab.Attributes.

The package content was changed to:

• 📂 analyzers/
  • 📂 dotnet/
    • 📂 roslyn3.11/
      • 📄 Jab.Roslyn3.dll
    • 📂 roslyn4.0/
      • 📄 Jab.dll
• 📂 build/
  • 📄 Jab.props
  • 📄 Jab.targets
• 📄 README.md

[skarllot]

All, done as suggested. Not sure about the naming of the compiler constants.

[pakrym]

Looks great, thank you!