bugraoz93 commented on PR #62073: URL: https://github.com/apache/airflow/pull/62073#issuecomment-3921885003
> @potiuk @jscheffl Please, _please_, don't use an incompatible format! > > The line > > `<!-- SPDX-License-Identifier: Apache-2.0 https://www.apache.org/licenses/LICENSE-2.0 -->` > > will break every license parser in existence. > > After the `SPDX-License-Identifier:`, an SPDX license expression is expected. See the [guidelines](https://spdx.github.io/using/license-id-in-source/), which in SPDXv2 used to be part of the specification that is an ISO standard [here](https://spdx.github.io/spdx-spec/v2.3/using-SPDX-short-identifiers-in-source-files/). > > If you need to put the license URL, please put it on a separate line. > > Personally, I don't believe it's needed, since `SPDX-License-Identifier: Apache-2.0` automatically means `https://spdx.org/licenses/Apache-2.0`. > > But in any case, don't add it to the same line! We should be able to use `Representing multiple licenses` and it looks like fit to format 🤔 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
