Skip to content

Conversation

@MattGal
Copy link
Member

@MattGal MattGal commented Apr 20, 2022

(we were restoring multiple before and this is undesirable for static analysis)

To double check:

@MattGal MattGal requested a review from ulisesh April 20, 2022 00:44
Copy link
Contributor

@ulisesh ulisesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MattGal MattGal merged commit 26bce56 into dotnet:main Apr 20, 2022
jonfortescue pushed a commit to jonfortescue/arcade that referenced this pull request Apr 21, 2022
* Force import of the "right" version so we always restore the version we ask for.

* Need to ignore for .proj since these don't have TFMs/restore correctly.
jonfortescue pushed a commit to jonfortescue/arcade that referenced this pull request Apr 22, 2022
* Force import of the "right" version so we always restore the version we ask for.

* Need to ignore for .proj since these don't have TFMs/restore correctly.
jonfortescue pushed a commit to jonfortescue/arcade that referenced this pull request Apr 22, 2022
* Force import of the "right" version so we always restore the version we ask for.

* Need to ignore for .proj since these don't have TFMs/restore correctly.
@premun
Copy link
Member

premun commented Oct 25, 2022

@MattGal just to double-check - should this change be specific to arcade?

I am asking because when onboarding Arcade, we ask people to copy this file over so it might not be desirable to have arcade specific things there.

@MattGal
Copy link
Member Author

MattGal commented Oct 25, 2022

@MattGal just to double-check - should this change be specific to arcade?

I am asking because when onboarding Arcade, we ask people to copy this file over so it might not be desirable to have arcade specific things there.

We need to be CG clean. If people copy and paste things outside the eng/common folder in Arcade, there are numerous examples of things that don't necessarily apply to other repos. If you have an approach that gets a green scan though that you feel is better for that "blind copy/paste" user group, I'm happy to approve that PR.

@premun
Copy link
Member

premun commented Oct 25, 2022

If people copy and paste things outside the eng/common folder in Arcade, there are numerous examples of things that don't necessarily apply to other repos

I understand that they just shouldn't copy any files and expect to adopt them, but we literally tell them to copy this one when starting a fresh repo:

image

What if people don't have the SystemSecurityCryptographyX509CertificatesVersion property available?

I think we could resolve this by putting another Directory.Build.props lower in Arcade's tree (in src/ or test/ or wherever we need it) and include the root DB.props from there.

MattGal added a commit to MattGal/arcade that referenced this pull request Oct 25, 2022
MattGal added a commit that referenced this pull request Oct 25, 2022
dotnet-bot pushed a commit to dotnet/dotnet that referenced this pull request Nov 1, 2022
Diff: https://github.com/dotnet/arcade/compare/c5dd6a1da2e6d9b3423ab809fcda8af2927a408b..6956787678e164e39ee646448e14353432d83c46

From: dotnet/arcade@c5dd6a1
To: dotnet/arcade@6956787

Commits:
  - Update dependencies from https://github.com/dotnet/arcade build 20221024.5 (#11399)
    dotnet/arcade@ec68425
  - Move the X509 Certificates version-pinning workaround up to a subdirectory.  (PR feedback from dotnet/arcade#9101 ) (#11402)
    dotnet/arcade@ec89a0f
  - Add feedback links to all the files (#11317)
    dotnet/arcade@758b26a
  - Allow backport workflow to be reusable (#11438)
    dotnet/arcade@6956787

[[ commit created by automation ]]
dotnet-bot pushed a commit to dotnet/dotnet that referenced this pull request Nov 7, 2022
Diff: https://github.com/dotnet/arcade/compare/c5dd6a1da2e6d9b3423ab809fcda8af2927a408b..80b6be47e1425ea90c5febffac119250043a0c92

From: dotnet/arcade@c5dd6a1
To: dotnet/arcade@80b6be4

Commits:
  - Update dependencies from https://github.com/dotnet/arcade build 20221024.5 (#11399)
    dotnet/arcade@ec68425
  - Move the X509 Certificates version-pinning workaround up to a subdirectory.  (PR feedback from dotnet/arcade#9101 ) (#11402)
    dotnet/arcade@ec89a0f
  - Add feedback links to all the files (#11317)
    dotnet/arcade@758b26a
  - Allow backport workflow to be reusable (#11438)
    dotnet/arcade@6956787
  - [main] Update dependencies from dotnet/roslyn (#11430)
    dotnet/arcade@c1a5de6
  - Fix IMAGE_DATA_DIRECTORY structure documentation url in comment (#11443)
    dotnet/arcade@bbe32bd
  - Suppress a couple warnings coming from automated scans (#11461)
    dotnet/arcade@f9e265e
  - Update dependencies from https://github.com/dotnet/arcade build 20221028.1 (#11470)
    dotnet/arcade@d4e96f1
  - Add scorecards for 2022-10-26 (#11448)
    dotnet/arcade@7607f96
  - Add One-pager for agentless helix (#11316)
    dotnet/arcade@5be5561
  - Refactor backport workflow, make it reusable (#11469)
    dotnet/arcade@a1a050e
  - Add release notes to arcade issue template (#11444)
    dotnet/arcade@d17c6da
  - Fix the target framework in the msi.proj now that net5.0 is unsupported by the sdk (#11502)
    dotnet/arcade@cabdcb2
  - Update dependencies from https://github.com/dotnet/arcade build 20221102.1 (#11497)
    dotnet/arcade@ee62d65
  - Update default msbuild for sdk tasks (#11508)
    dotnet/arcade@80b6be4

[[ commit created by automation ]]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants