It would be better to name it "artifacts"

Done :)

Could you please explain what is the difference between DotNetCoreRestore and NuGetRestore?

NuGetRestore uses the nuget cli instead of dotnet. Apologies I've only really written cake scripts for full framework projects before so I used what I know! I've updated to use DotNetRestore

Please always clean outputNupkg, obj and bin directories clean if it is a local build

Done

No check if it is a local build

Can we use DotNetCoreBuild here?

Yes. Have changed.

You don't actually need to hardcode tha path, just use something like GetFiles(@"**/*.nupkg")

No longer relevant now I've changed to DotNetPack

Now, it will generate package on each build. We can remove this line and add DotNetCorePack task to build.cake to generate only if it is needed

Done

what do you think about this way:

Task("Clean")
    .Does(() =>
    {
        CleanDirectory(artifactsDir);

        if(BuildSystem.IsLocalBuild)
        {
            CleanDirectories(GetDirectories("./**/obj") + GetDirectories("./**/bin"));
        }
    });

Yeah that's a good way 👍

By default DotNetCoreBuild will run nuget restore command. As it is currently implemented, we run "Restore-NuGet-Packages" task before "Build". So, we don't actually need to run restore command again. You can do it in this way:

DotNetCoreBuild(projectFile, new DotNetCoreBuildSettings
{
	Configuration = configuration,
	NoRestore = true
});

it will decrease build time. yes it is only a few milliseconds, but sometimes milliseconds is important :)

A fair point. Changed.

The same here. DotNetCorePack runs nuget restore command and build command. In this task, we have already restored and built a project. Try this one:

var settings = new DotNetCorePackSettings
{
	Configuration = configuration,
	NoRestore = false,
	NoBuild = true,
	OutputDirectory = artifactsDir
};

DotNetCorePack(projectFile, settings);

Done 👍