这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@brettfo
Copy link
Member

@brettfo brettfo commented Jul 19, 2019

Copying from dotnet/fsharp, adding a CI step to verify we don't unnecessarily rebuild the node dependencies. The process is to build the solution, gather the modification timestamp from some sentinel files, build again, and verify the timestamps haven't changed, indicating a rebuild.

Check the $FileList array in UpToDate.ps1 for the list of sentinel files that we check.

@brettfo
Copy link
Member Author

brettfo commented Jul 19, 2019

Initial run shows a failure (which is good). Will amend PR with fix before merging.

1 of 3 files were re-built.
    D:\a\1\s\MLS.Agent\wwwroot\css\trydotnet.css
##[error]PowerShell exited with code '1'.

@brettfo brettfo changed the title [WIP] add up-to-date CI check add up-to-date CI check Jul 19, 2019
@brettfo
Copy link
Member Author

brettfo commented Jul 19, 2019

After fixing the trydotnet.css issue and triggering a failure on anything changing under wwwroot/, the output now ends with:

0 of 68 files were re-built.

@brettfo brettfo merged commit 7a643a1 into dotnet:master Jul 22, 2019
@brettfo brettfo deleted the up-to-date branch July 22, 2019 20:25
akshita31 pushed a commit to akshita31/try that referenced this pull request Jul 29, 2019
* add up-to-date CI check

* fix target inputs typo

* check all files under `wwwroot/` during up-to-date check
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.

2 participants