tisonkun commented on PR #543: URL: https://github.com/apache/pulsar-site/pull/543#issuecomment-1562151294
Isn't it general that "the PR is outdated, merge master to see if it fixes your problem"? We follow this pattern in the main repo for fixing flaky tests. I agree that I may upgrade all PRs to catch up the newest version but that should not be a requirement. Two possible points here: 1. Make an online preview - If we use Vercel, it can be done smoothly. But we're deploy with Apache server, so it at least requires some development. 2. CI to verify - Yes, the CI setting can build this PR. You local environment mismatch - it's not a serious problem, right? What if you build master Pulsar with Java 6 or Java 21 EA and encounter problem? Even that the requirement of Node version is fixed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
