Skip to content

Conversation

@parfeon
Copy link
Contributor

@parfeon parfeon commented Oct 18, 2021

fix(api-client): fix hostname encoding

Fix issue because of which it is impossible to set hostaname with specific port number.

test(contract): add acceptance tests

Add contract based acceptance tests.

Fix `artifacts` field content and align with specs requirement.

docs(.pubnub.yml): fix missing feature entries

Update features matrix with information about auth token parse and set features.
Update list of codeowners of `.pubnub.yml` file.
Add contract based acceptance tests.

fix(api-client): fix hostname encoding

Fix issue because of which it is impossible to set hostaname with specific port number.
Remove feature files from repository and let GitHub Actions handle them.
Make it possible for CI/CD to pass when beta contracts fail.
@parfeon parfeon added priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: test This PR contains new tests for existing functionality or fixes to existing tests. labels Oct 18, 2021
@parfeon parfeon self-assigned this Oct 18, 2021
Use newly exposed environment variable in attempt to solve show build settings timeout.
Add dependencies pulled out by CocoaPods to cache for build speedup.
Add gems installed by bundler to cache for build speedup.
Use bundler to install and run fastlane, to make GitHub Action cache useful.
Fix path which is used to store acceptance tests result for artifacts upload.
@parfeon parfeon merged commit bcb41ad into master Nov 3, 2021
@parfeon parfeon deleted the test/SDK-1350/add-acceptance-tests branch November 3, 2021 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: test This PR contains new tests for existing functionality or fixes to existing tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants