Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: adding experiment function for rum bundler APIs #264

Merged
merged 7 commits into from
Jun 20, 2024

Conversation

rpapani
Copy link
Contributor

@rpapani rpapani commented Jun 13, 2024

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues

Thanks for contributing!

Copy link

This PR will trigger a minor release when merged.

@rpapani rpapani marked this pull request as ready for review June 16, 2024 23:41
@rpapani rpapani requested review from ekremney and ramboz June 16, 2024 23:41
Comment on lines 33 to 37
it('crunches experiment data', async () => {
const experimentsResult = experiment.handler(bundles.rumBundles);
console.log(JSON.stringify(experimentsResult));
// expect(expected404Result).to.eql(notfoundResult);
});
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather not commit a test, or clearly mark it skipped, if it's not complete. don't want to give the false impression that this is actually tested down the road if we don't have any expectation yet 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

committed in hurry, during debug session with Ekrem. Fixed the tests properly now.

Copy link
Member

@ekremney ekremney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great stuff 👍

@rpapani rpapani merged commit b1a076d into main Jun 20, 2024
9 checks passed
@rpapani rpapani deleted the rum-bundler-experiment branch June 20, 2024 14:13
adobe-bot pushed a commit that referenced this pull request Jun 20, 2024
# [@adobe/spacecat-shared-rum-api-client-v2.1.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.0.3...@adobe/spacecat-shared-rum-api-client-v2.1.0) (2024-06-20)

### Features

* adding experiment function for rum bundler APIs ([#264](#264)) ([b1a076d](b1a076d))
@adobe-bot
Copy link

🎉 This PR is included in version @adobe/spacecat-shared-rum-api-client-v2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants