Steven Edwards

icon indicating a website link https://resumes.indeed.com/resume/6721525c2880dd6d icon indicating an email [email protected]

icon location Taylorsville, NC icon location Coder since '94. Go since 1.1. Building tools (cli, tui, emacs, backend) is what excites me these days. Will consider remote offers.

Results 92 comments of Steven Edwards

Remove else statements from if/return blocks.

> I don't currently think... > > > deconstructed into smaller structs > > ...is a good idea, since this somewhat breaks api or intellisense of top level fields. >...

Remove else statements from if/return blocks.

Will this be accepted if I remove the sorted fields?

Remove else statements from if/return blocks.

> from my perspective this would increase a chance of a merge by 400%, but let's hear wolfogres opinion > > _In the past 2021/22 such kind of PR would...

Remove else statements from if/return blocks.

The revert is complete along with a cleanup of findGitSlug if you're interested.

Incorporate Debug, Warn, Error logging.

The OS/X check failed.

Incorporate Debug, Warn, Error logging.

This PR works, but `act -q` is closer to what I wanted.

Incorporate Debug, Warn, Error logging.

@mcascone This should be available tomorrow at [this fork](https://github.com/actions-oss/act-cli/).

act shouldn't complain when there's no remote repo

This used to work but was removed. https://github.com/nektos/act/compare/9085c833bf2b68e90c022e70912b3c0c4499bdbc..ace4cd47c7f099864866b1f60e064fecde7f36ea#diff-c057d66dc9657d8428e290c69871596e2b567bb8fecad62a99cab54398131a84L679

act shouldn't complain when there's no remote repo

The main issue is `getGithubContext` is called in multiple places, so the error repeats over and over.

act shouldn't complain when there's no remote repo

This bypasses the remote warning introduced in #2169