Skip to content

[0.64] Pull in upstream fixes to expose hover props on Pressable #884 #885

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

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
c2b2e2f
add pull yml
Saadnajmi Mar 22, 2021
ab88c7d
Merge pull request #1 from microsoft/master
Saadnajmi Apr 2, 2021
7a9006b
match handleOpenURLNotification event payload with iOS (#755) (#2)
pull[bot] Apr 21, 2021
497aa72
Merge pull request #3 from microsoft/master
Saadnajmi Apr 28, 2021
1c81e5b
Merge pull request #4 from microsoft/master
Saadnajmi May 6, 2021
28aed35
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi May 20, 2021
62dc473
Merge branch 'master' of github.com:Saadnajmi/react-native-macos
Saadnajmi May 20, 2021
93c7296
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi May 21, 2021
780b2b7
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Jun 11, 2021
99d5182
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Aug 22, 2021
09e872d
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Sep 3, 2021
9a25530
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Oct 6, 2021
d82a01b
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Oct 12, 2021
d9faa3a
[pull] master from microsoft:master (#11)
pull[bot] Oct 14, 2021
39bd488
Merge branch 'master' of github.com:microsoft/react-native-macos
Saadnajmi Oct 21, 2021
8960e85
Some more changes for bringing up RN64 in devmain Android (#861)
mganandraj Oct 22, 2021
f9f7f8d
Fix for submit button disappearing bug in comments (#862)
msgharpu Oct 25, 2021
78a879c
fix: ensure scripts always have Unix newlines (#860)
tido64 Oct 26, 2021
e1ea6d2
Post NSAccessibilityLayoutChangedNotification when we call setAccessi…
amgleitman Oct 27, 2021
4baf0f3
Merge pull request #872 from amgleitman/amgleitman/macos-accessibilit…
amgleitman Oct 28, 2021
a186688
Fix for publishing from stable branches (#867)
acoates-ms Oct 28, 2021
f8cea11
Fixes for publishing from stable branches
acoates-ms Nov 1, 2021
ce8abc5
nullable compat
HeyImChris Nov 4, 2021
80160a9
undo last nullable check
HeyImChris Nov 4, 2021
ea0f1ac
Merge pull request #879 from acoates-ms/updpodlock
acoates-ms Nov 4, 2021
8937e55
Rename directory for ship .so files to be ship instead of release
rasaha91 Nov 4, 2021
f4aab2d
Nullable compatibility (#882)
HeyImChris Nov 4, 2021
0818f75
Expose Pressability Hover config props in Pressable (#32405)
Saadnajmi Oct 26, 2021
718c3f2
Extra followup changes
Saadnajmi Nov 4, 2021
425569a
remove some extra fork differences from change
Saadnajmi Nov 4, 2021
596e012
Merge pull request #886 from microsoft/rasaha
rasaha91 Nov 5, 2021
8b72f08
Bump tar from 4.4.13 to 4.4.19 in /packages/react-native-macos-init
dependabot[bot] Nov 5, 2021
2314904
Merge pull request #839 from microsoft/dependabot/npm_and_yarn/packag…
acoates-ms Nov 5, 2021
5915c64
Bump hosted-git-info in /packages/react-native-macos-init
dependabot[bot] Nov 5, 2021
01357c9
Merge pull request #838 from microsoft/dependabot/npm_and_yarn/packag…
acoates-ms Nov 5, 2021
58a6463
Add back type
Saadnajmi Nov 7, 2021
a23a1f1
update podfile
Saadnajmi Nov 7, 2021
4c86ca6
Merge branch 'master' of https://github.com/microsoft/react-native-macos
HeyImChris Nov 8, 2021
4545eca
nonnull option
HeyImChris Nov 8, 2021
21e2955
Merge pull request #896 from HeyImChris/nullabilityCheck
acoates-ms Nov 9, 2021
8b8f01b
add a nullable check (#898)
HeyImChris Nov 10, 2021
2bb6d93
Disable dev mode on ship builds (#888)
HeyImChris Nov 11, 2021
f88f01a
ci: add a pipeline for consuming react-native-macos (#878)
tido64 Nov 12, 2021
02d8438
Merge branch 'master' of github.com:microsoft/react-native-macos into…
Saadnajmi Dec 7, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 18 additions & 4 deletions .ado/android-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,24 @@ jobs:
SYSTEM_ACCESSTOKEN: $(System.AccessToken)
githubApiToken: $(githubApiToken)

- task: CmdLine@2
displayName: gradlew clean
inputs:
script: ./gradlew clean
# In RN64 "clean" task tries to run the build again. It is reproing in 0.64-stable branch but fixed in 0.65-stable.
# I can't zero on the exact cause .. but turns out the issue is with rntester clean task.
#
# mganandraj@RNDroid:~/github/rnm-64$ ./gradlew :packages:rn-tester:android:app:clean --dry-run
# :packages:react-native-codegen:android:buildCodegenCLI SKIPPED
# :packages:rn-tester:android:app:generateCodegenSchemaFromJavaScript SKIPPED
# :packages:rn-tester:android:app:generateCodegenArtifactsFromSchema SKIPPED
# :ReactAndroid:extractAARHeaders SKIPPED
# :ReactAndroid:extractJNIFiles SKIPPED
# :ReactAndroid:generateCodegenSchemaFromJavaScript SKIPPED
# :ReactAndroid:generateCodegenArtifactsFromSchema SKIPPED
# :ReactAndroid:prepareBoost SKIPPED
# :ReactAndroid:createNativeDepsDirectories SKIPPED
# ..
# - task: CmdLine@2
# displayName: gradlew clean
# inputs:
# script: ./gradlew clean

- task: PublishBuildArtifacts@1
displayName: 'Publish final artifacts'
Expand Down
97 changes: 97 additions & 0 deletions .ado/apple-integration.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,97 @@
name: Integrate $(Date:yyyyMMdd).$(Rev:.r)
variables:
- template: variables/mac.yml
trigger:
branches:
include:
- master
- '*-stable'
paths:
exclude:
- '*.md'
pr:
branches:
include:
- master
- '*-stable'
paths:
exclude:
- '*.md'
jobs:
- job: react_native_test_app
displayName: react-native-test-app
pool:
vmImage: $(VmImage)
demands: ['npm', 'sh', 'xcode']
workspace:
clean: all
timeoutInMinutes: 60
cancelTimeoutInMinutes: 5
steps:
- template: templates/apple-node-setup.yml
- template: templates/apple-xcode-select.yml
parameters:
slice_name: $(slice_name)
xcode_version: $(xcode_version)
- bash: |
echo "##vso[task.setvariable variable=package_version]$(cat package.json | jq .version | awk '{ print substr($0, 2, length($0) - 2) }')"
echo "##vso[task.setvariable variable=react_version]$(cat package.json | jq .peerDependencies.react)"
echo "##vso[task.setvariable variable=rncli_version]$(cat package.json | jq '.dependencies."@react-native-community/cli"')"
displayName: 'Determine react-native-macos version'
- bash: |
npm pack
displayName: 'Pack react-native-macos'
- bash: |
git clone --progress https://github.com/microsoft/react-native-test-app.git
displayName: Checkout react-native-test-app
- bash: |
set -eo pipefail
cat package.json |
jq '.devDependencies["react"] = $(react_version)' |
jq '.devDependencies["react-native"] = "^0.64"' |
jq '.devDependencies["react-native-macos"] = "../react-native-macos-$(package_version).tgz"' |
jq 'del(.devDependencies["@react-native-community/cli"])' |
jq 'del(.devDependencies["@react-native-community/cli-platform-android"])' |
jq 'del(.devDependencies["@react-native-community/cli-platform-ios"])' |
jq 'del(.devDependencies["react-native-windows"])' > .package.json
mv .package.json package.json
cat package.json | jq .devDependencies
displayName: Modify react-native-test-app dependencies
workingDirectory: react-native-test-app
- bash: |
set -eo pipefail
cat package.json |
jq '.devDependencies["@react-native-community/cli"] = $(rncli_version)' |
jq '.devDependencies["@react-native-community/cli-platform-android"] = $(rncli_version)' |
jq '.devDependencies["@react-native-community/cli-platform-ios"] = $(rncli_version)' |
jq '.devDependencies["react"] = $(react_version)' |
jq '.devDependencies["react-native"] = "^0.64"' |
jq '.devDependencies["react-native-macos"] = "../../react-native-macos-$(package_version).tgz"' |
jq 'del(.devDependencies["react-native-windows"])' > .package.json
mv .package.json package.json
cat package.json | jq .devDependencies
displayName: Modify example app dependencies
workingDirectory: react-native-test-app/example
- bash: |
yarn --no-immutable
displayName: Install npm dependencies
workingDirectory: react-native-test-app
- bash: |
yarn build:macos || yarn build:macos
displayName: Bundle JavaScript
workingDirectory: react-native-test-app/example
- bash: |
pod install --project-directory=macos
displayName: Install Pods
workingDirectory: react-native-test-app/example
- bash: |
set -eo pipefail
../scripts/xcodebuild.sh macos/Example.xcworkspace build
displayName: Build Intel
workingDirectory: react-native-test-app/example
- bash: |
set -eo pipefail
../scripts/xcodebuild.sh macos/Example.xcworkspace clean
../scripts/xcodebuild.sh macos/Example.xcworkspace build ARCHS=arm64
displayName: Build ARM
workingDirectory: react-native-test-app/example
2 changes: 1 addition & 1 deletion .ado/gitTagRelease.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ function doPublish() {
exec(`git config --global user.email "[email protected]"`);
exec(`git config --global user.name "React-Native Bot"`);

exec(`git add ${pkgJsonPath}`);
exec(`git add .`);
exec(`git commit -m "Applying package update to ${releaseVersion} ***NO_CI***"`);
exec(`git tag v${releaseVersion}`);
exec(`git push origin HEAD:${tempPublishBranch} --follow-tags --verbose`);
Expand Down
2 changes: 1 addition & 1 deletion .ado/npmOfficePack.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ function doPublish(fakeMode) {
if (!onlyTagSource) {
// -------- Generating Android Artifacts with JavaDoc
const depsEnvPrefix = "REACT_NATIVE_BOOST_PATH=" + path.join(process.env.BUILD_SOURCESDIRECTORY, "build_deps");
const gradleCommand = path.join(process.env.BUILD_SOURCESDIRECTORY, "gradlew") + " installArchives";
const gradleCommand = path.join(process.env.BUILD_SOURCESDIRECTORY, "gradlew") + " installArchives -Pparam=\"excludeLibs\"";
exec( depsEnvPrefix + " " + gradleCommand );

// undo uncommenting javadoc setting
Expand Down
39 changes: 35 additions & 4 deletions .ado/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,14 @@ trigger:

pr: none

variables:
- template: variables/mac.yml

jobs:
- job: RNGithubNpmJSPublish
displayName: React-Native GitHub Publish to npmjs.org
pool:
vmImage: vs2017-win2016
vmImage: $(VmImage)
timeoutInMinutes: 90 # how long to run the job before automatically cancelling
cancelTimeoutInMinutes: 5 # how much time to give 'run always even if cancelled tasks' before killing them
steps:
Expand All @@ -47,6 +50,8 @@ jobs:
displayName: Set dist-tag to v0.x-stable
condition: and(ne(variables['Build.SourceBranchName'], 'master'), ne(variables['Build.SourceBranchName'], variables.latestStableBranch))

- template: templates/apple-node-setup.yml

- task: CmdLine@2
displayName: yarn install
inputs:
Expand All @@ -56,17 +61,37 @@ jobs:
displayName: Bump stable package version
inputs:
script: node .ado/bumpFileVersions.js
condition: ne(variables['Build.SourceBranchName'], 'master')
condition: and(succeeded(), ne(variables['Build.SourceBranchName'], 'master'))

- task: CmdLine@2
displayName: pod update React-TurboModuleCxx-RNW
inputs:
script: |
cd packages/rn-tester
pod update React-TurboModuleCxx-RNW
condition: and(succeeded(), ne(variables['Build.SourceBranchName'], 'master'))

- task: CmdLine@2
displayName: Bump canary package version
inputs:
script: node scripts/bump-oss-version.js --nightly
condition: eq(variables['Build.SourceBranchName'], 'master')
condition: and(succeeded(), eq(variables['Build.SourceBranchName'], 'master'))

# Publish will fail if package.json is marked as private
- task: CmdLine@2
displayName: Remove workspace config from package.json
inputs:
script: node .ado/removeWorkspaceConfig.js

- script: npm publish --tag $(npmDistTag) --registry https://registry.npmjs.org/ --//registry.npmjs.org/:_authToken=$(npmAuthToken)
displayName: Publish react-native-macos to npmjs.org

# Put the private flag back so that the removal does not get committed by the tag release step
- task: CmdLine@2
displayName: Restore package.json workspace config
inputs:
script: node .ado/restoreWorkspaceConfig.js

- task: CmdLine@2
displayName: 'Tag published release'
inputs:
Expand All @@ -76,7 +101,7 @@ jobs:
BUILD_SOURCEBRANCH: $(Build.SourceBranch)
SYSTEM_ACCESSTOKEN: $(System.AccessToken)
githubApiToken: $(githubApiToken)
condition: ne(variables['Build.SourceBranchName'], 'master')
condition: and(succeeded(), ne(variables['Build.SourceBranchName'], 'master'))


- job: RNMacOSInitNpmJSPublish
Expand Down Expand Up @@ -137,6 +162,12 @@ jobs:
inputs:
script: node .ado/bumpOfficeFileVersions.js

# Publish will fail if package.json is marked as private
- task: CmdLine@2
displayName: Remove workspace config from package.json
inputs:
script: node .ado/removeWorkspaceConfig.js

- task: CmdLine@2
displayName: gradlew installArchives
inputs:
Expand Down
3 changes: 3 additions & 0 deletions .ado/removeWorkspaceConfig.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
// @ts-check
const {removeWorkspaceConfig} = require('./versionUtils');
removeWorkspaceConfig();
3 changes: 3 additions & 0 deletions .ado/restoreWorkspaceConfig.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
// @ts-check
const {restoreWorkspaceConfig} = require('./versionUtils');
restoreWorkspaceConfig();
21 changes: 19 additions & 2 deletions .ado/templates/android-build-office.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,29 @@ steps:
inputs:
script: yarn install --frozen-lockfile

# scripts/bump-oss-version.js
#
# update the following files to the commit id.
# package.json (-> npm package version)
# gradle.properties (-> aar version)
# gradle.properties (-> aar version)
#
# The following files are updated for runtime version checking.
# ReactAndroid/src/main/java/com/facebook/react/modules/systeminfo/ReactNativeVersion.java
# React/Base/RCTVersion.m
# ReactCommon/cxxreact/ReactNativeVersion.h
# Libraries/Core/ReactNativeVersion.js
#
# --nightly => version = `0.0.0-${currentCommit.slice(0, 9)}`;
# When on master branch or non-stable branch.
- task: CmdLine@2
displayName: Bump canary package version
inputs:
script: node scripts/bump-oss-version.js --nightly
condition: eq(variables['Build.SourceBranchName'], 'master')

condition: or(eq(variables['Build.SourceBranchName'], 'master'), not(contains(variables['Build.SourceBranchName'], '-stable')))

# TODO: We don't seem to be running bump-oss-version.js for stable branches, hence we would end up publishing using the values in the repository.

- task: CmdLine@2
displayName: nuget restore
inputs:
Expand Down
2 changes: 1 addition & 1 deletion .ado/templates/apple-droid-node-patching.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ steps:
- task: CmdLine@2
displayName: Apply Android specific patches for Office consumption
inputs:
script: node $(System.DefaultWorkingDirectory)/android-patches/bundle/bundle.js patch $(System.DefaultWorkingDirectory) Build OfficeRNHost V8 Focus --patch-store $(System.DefaultWorkingDirectory)/android-patches/patches --log-folder $(System.DefaultWorkingDirectory)/android-patches/logs --confirm ${{ parameters.apply_office_patches }}
script: node $(System.DefaultWorkingDirectory)/android-patches/bundle/bundle.js patch $(System.DefaultWorkingDirectory) Build OfficeRNHost V8 Focus MAC ImageColor --patch-store $(System.DefaultWorkingDirectory)/android-patches/patches --log-folder $(System.DefaultWorkingDirectory)/android-patches/logs --confirm ${{ parameters.apply_office_patches }}
26 changes: 23 additions & 3 deletions .ado/versionUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
const fs = require("fs");
const path = require("path");
const semver = require('semver');
const {execSync} = require('child_process');

const pkgJsonPath = path.resolve(__dirname, "../package.json");
let publishBranchName = '';
Expand Down Expand Up @@ -41,15 +42,34 @@ function updateVersionsInFiles(patchVersionPrefix) {
}

pkgJson.version = releaseVersion;
fs.writeFileSync(pkgJsonPath, JSON.stringify(pkgJson, null, 2));
console.log(`Updating package.json to version ${releaseVersion}`);
console.log(`Bumping files to version ${releaseVersion}`);
execSync(`node ./scripts/bump-oss-version.js --rnmpublish ${releaseVersion}`, {stdio: 'inherit', env: process.env});

return {releaseVersion, branchVersionSuffix};
}

const workspaceJsonPath = path.resolve(require('os').tmpdir(), 'rnpkg.json');

function removeWorkspaceConfig() {
let {pkgJson} = gatherVersionInfo();
fs.writeFileSync(workspaceJsonPath, JSON.stringify(pkgJson, null, 2));
delete pkgJson.private;
delete pkgJson.workspaces;
fs.writeFileSync(pkgJsonPath, JSON.stringify(pkgJson, null, 2));
console.log(`Removing workspace config from package.json to prepare to publish.`);
}

function restoreWorkspaceConfig() {
let pkgJson = JSON.parse(fs.readFileSync(workspaceJsonPath, "utf8"));
fs.writeFileSync(pkgJsonPath, JSON.stringify(pkgJson, null, 2));
console.log(`Restoring workspace config from package.json`);
}

module.exports = {
gatherVersionInfo,
publishBranchName,
pkgJsonPath,
removeWorkspaceConfig,
restoreWorkspaceConfig,
updateVersionsInFiles
}
6 changes: 6 additions & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# Ensure scripts always have Unix newlines, even on Windows.
*.command text eol=lf
*.sh text eol=lf
# Windows files should use crlf line endings
# https://help.github.com/articles/dealing-with-line-endings/
*.bat text eol=crlf
Loading