Skip to content

Initial cucumber-java-playwright BrowserStack sample#1

Open
Jimesh-browserstack wants to merge 4 commits intomainfrom
master
Open

Initial cucumber-java-playwright BrowserStack sample#1
Jimesh-browserstack wants to merge 4 commits intomainfrom
master

Conversation

@Jimesh-browserstack
Copy link
Copy Markdown

Summary

Replaces the placeholder main branch content with the full cucumber-java-playwright BrowserStack sample repo.

  • Adds pom.xml, browserstack.yml, LICENSE, .gitignore
  • Adds sample tests: RunCucumberTest + RunCucumberLocalTest with stepdefs and .feature files for e2e and Local flows
  • Adds testng.xml / testngLocal.xml runners
  • Adds .github/workflows/cucumber-workflow-run.yml (moves the existing Semgrep workflow alongside it)
  • Moves CODEOWNERS to .github/CODEOWNERS (was root-level) and expands ownership: asi-devs owns workflows, automate-public-repos owns everything else
  • Replaces the 2-line stub README with full customer-facing docs

Note on history

master was pushed as a root commit, so a --strategy=ours merge of origin/main was used to link histories. This preserves all of master's tree while creating a common ancestor, which is why the PR diff shows the stub-to-full replacement cleanly.

Test plan

  • Verify cucumber-workflow-run.yml runs on merge
  • Verify Semgrep workflow still passes
  • Confirm CODEOWNERS routes reviews correctly on future PRs
  • After merge, consider making master the default branch (or delete master and continue from main)

Comment thread .github/workflows/cucumber-workflow-run.yml Fixed
Jimesh-browserstack and others added 2 commits April 24, 2026 14:04
Keeps the root-level CODEOWNERS file so merging to main does not remove it.
.github/CODEOWNERS takes precedence for GitHub's ownership routing.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Addresses CodeQL finding: workflow did not limit GITHUB_TOKEN permissions.
- Top-level: contents: read
- Job-level: checks: write (required by actions/github-script calls to
  github.rest.checks.create)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Jimesh-browserstack Jimesh-browserstack self-assigned this Apr 24, 2026
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.

4 participants