Reevaluate how we embed assets for SOS-containing packages

オープン
#5,566 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
cpp

調査の方向性

まず、単体テスト用の依存関係アセットがどのように配置されているか、また Copy task がどこで部分的な publish をエミュレートしているかを追跡します。そのフローを pull request 5559 の回避策と比較します。完了条件は、プロジェクトが必要に応じて publish され、重複したアセットが衝突しなくなり、必要なテスト構成パスが修正されていることです。

索引モデルが issue の本文から書いたものです。

説明

enhancement Infrastructure

Global tools were reworked to use publish as a precursor to package. The way we get assets of dependencies for projects for unit testing breaks this assumption since we create the layout manually using the Copy task. This gets the packaging logic confused and we have a colision of assets coming from two places, even though they are the same asset.

This was worked around in https://github.com/dotnet/diagnostics/pull/5559, but the real long term fix is to stop using Copy long term and just publish projects as needed instead of trying to emulate the behavior of partial publish. This might need some fixups in the config paths for tests, but is less likely to break on the long run.

主要言語
C++
スター
1.3k
フォーク
407
平均マージ
21時間 5分
マージ済み PR(30日)
78

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートなし
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

dotnet/diagnostics のほかの issue

dotnet/diagnostics の issue をすべて見る

似ている issue

C++ の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。